https://bugzilla.redhat.com/show_bug.cgi?id=1421413 --- Comment #10 from Marek Cermak <macermak@xxxxxxxxxx> --- Thanks, Frank, for quick response and Jaroslav, for addressing another issues and additional comments. (In reply to Jaroslav Škarvada from comment #2) > (In reply to Marek Cermak from comment #1) > > - Consider putting content of Web App into /usr/share/%{name} instead of > > /var/www/ > > > Probably OK, bug 1028722. Checked the bug 1028722 /var/www is considered acceptable, no need to change it. (In reply to Jaroslav Škarvada from comment #4) > (In reply to Marek Cermak from comment #1) > > file /etc/cron.d is not owned by any package > > False positive, the package requires crontabs which requires /etc/cron.d, > which is provided by cronie, i.e. the dependency is fulfilled. I've checked the /etc/cron* , it is indeed false positive. (In reply to Jaroslav Škarvada from comment #3) > Also I think the license tag should be GPLv3+, not GPLv3. Agreed, it would be wiser to specify GPLv3+ in the license tag since licensecheck states the 6 files to be GPLv3 or later. (In reply to Jaroslav Škarvada from comment #7) > (In reply to Marek Cermak from comment #1) > > [!]: Fully versioned dependency in subpackages if applicable. > > Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in > > sysusage-common , sysusage-httpd , sysusage-rsysusage > > False positive, noarch packages, no %{?_isa} is needed. Right. I beat the gun there. -- 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@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
