Skip to content

Update Fedora install documentation, and avoid a use of an undefined variable #38

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kgaillot wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update Fedora install documentation, and avoid a use of an undefined variable #38

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
29e33bd
update Fedora install instructions in README.md
Sep 20, 2024
dd386a1
avoid use of undefined variable in error message
Sep 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,11 @@ libmail-mbox-messageparser-perl libwww-perl unzip
- For PostgreSQL: `libdbd-pg-perl`
- For Oauth2: `liblwp-protocol-https-perl libencode-perl libtime-piece-mysql-perl`

## on Fedora (Fedora 23)
## on Fedora (Fedora 40)

```
sudo dnf install perl-File-MimeInfo perl-Mail-IMAPClient perl-MIME-tools perl-XML-Simple perl-DBI \
sudo dnf install perl-File-MimeInfo perl-Mail-IMAPClient \
perl-Mail-Mbox-MessageParser perl-MIME-tools perl-XML-Simple perl-DBI \
perl-Socket6 perl-PerlIO-gzip perl-libwww-perl unzip
```

Expand Down Expand Up @@ -263,4 +264,4 @@ Thanks to [all who contributed](https://github.com/userjack6880/Open-Report-Pars

# Licenses and Copyrights

Copyright © 2023 John Bradley (userjack6880), Copyright © 2016 TechSneeze.com, Copyright © 2012 John Bieling. Open Report Parser is released under GNU GPLv3. See `LICENSE`.
Copyright © 2023 John Bradley (userjack6880), Copyright © 2016 TechSneeze.com, Copyright © 2012 John Bieling. Open Report Parser is released under GNU GPLv3. See `LICENSE`.
4 changes: 2 additions & 2 deletions report-parser.pl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -950,7 +950,7 @@ sub getXMLFromZip {
}
}
else {
warn "$scriptname: Could not find an <$filename>! \n";
warn "$scriptname: No ZIP file name specified! \n";
}

return $xml;
Expand Down Expand Up @@ -1830,4 +1830,4 @@ sub db_column_info {
$columns{$column} = $db_info->{$column}{$dbx{column_info_type_col}};
}
return %columns;
}
}