[Bug 1233279] Review Request: perl-Log-Agent - Logging agent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1233279



--- Comment #1 from Petr Šabata <psabata@xxxxxxxxxx> ---
Package the README file as %doc rather then %license.  The %license macro is
reserved for license texts so they get installed even if documentation is
disabled.  This README file doesn't provide, from the licensing point of view, 
anything what the RPM License tag or the modules themselves don't.

Consider packaging Log-Agent-Rotate for the optional log rotating features.

The following build-time deps aren't needed since the files requiring them
aren't tested:
 - Mail::Mailer
 - Sys::Syslog

The patch is fine.

So is the license.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]