needinfo canceled: [Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
Product: Fedora Version: rawhide Component: Package Review Package Review has canceled Package Review 's request for Devrim Gündüz 's needinfo: Bug 1254799: Review Request: pgbadger - A fast PostgreSQL log analyzer https://bugzilla.redhat.com/show_bug.cgi?id=1254799 --- Comment #7 from Package Review --- This is an automatic action taken by review-stats script. The ticket submitter failed to clear the NEEDINFO flag in a month. As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews we consider this ticket as DEADREVIEW and proceed to close it. ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1254799 --- Comment #5 from Devrim GÜNDÜZ --- (In reply to Jakub Jedelsky from comment #3) > Hi, > > there are missing dependencies in the spec file. You have to add (at least): > > BuildRequires: perl > BuildRequires: perl-ExtUtils-MakeMaker > Requires: perl Thanks, done, and also updated to 9.1: Spec URL: https://devrim.fedorapeople.org/pgbadger/pgbadger.spec SRPM URL: https://devrim.fedorapeople.org/pgbadger/pgbadger-9.1-1.fc25.src.rpm Regards, Devrim -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1254799 --- Comment #4 from Jakub Jedelsky --- Another observation - pgBadger uses and includes(!) bootstrap.css, some fancy fonts and JS libraries (jqplot, jquery). 1. please check, if it's possible to bundle them together or is better to split it somehow. I'm not the expert for that. There is some note, that fonts should be packaged standalone (font Awesome is in fedora already, so you can add it as a dependency and drop it from package). https://fedoraproject.org/wiki/Packaging:Guidelines#Avoid_bundling_of_fonts_in_other_packages 2. It looks that these css, js libraries have different licenses, so you should mention them with License tag. https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios for more details. Thanks for the work, I'm looking forward to have the package in repos :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1254799 Jakub Jedelsky changed: What|Removed |Added CC||jakub.jedel...@gmail.com --- Comment #3 from Jakub Jedelsky --- Hi, there are missing dependencies in the spec file. You have to add (at least): BuildRequires: perl BuildRequires: perl-ExtUtils-MakeMaker Requires: perl -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1254799 --- Comment #2 from Devrim GÜNDÜZ --- (In reply to Eduardo Mayorga from comment #1) > SRPM URL returns 404 Not Found. Updated to 9.0: Spec URL: https://devrim.fedorapeople.org/pgbadger/pgbadger.spec SRPM URL: https://devrim.fedorapeople.org/pgbadger/pgbadger-9.0-1.f24.src.rpm Thanks! Regards, Devrim -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1254799] Review Request: pgbadger - A fast PostgreSQL log analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1254799 Eduardo Mayorga changed: What|Removed |Added CC||e...@mayorgalinux.com --- Comment #1 from Eduardo Mayorga --- SRPM URL returns 404 Not Found. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
