Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=466223
--- Comment #2 from Paul Howarth paul@city-fan.org 2008-10-09 18:48:38 EDT --- (In reply to comment #1)
I don't really object to it, but I think the first paragraph of %description is content-free in the context of a package description.
OK, I stripped that paragraph and added a few words about logging to the second paragraph.
/usr/bin/one-liner probably should not be shipped; it seems completely pointless, has a generic name, and is already shipped as documentation.
I initially %exclude-d this for those very reasons but changed my mind as I thought it seemed a handy way of running methods from the module quickly from the command line. You're right though, it's very generic and doesn't seem to documented as a feature of the module so I've %exclude-d it again.
The package review process needs reviewers! If you haven't done any package reviews recently, please consider doing one.
Getting the sendmail merge review moving again is on my to-do list...
Revised package: http://www.city-fan.org/~paul/extras/perl-Sysadm-Install/perl-Sysadm-Install... http://www.city-fan.org/~paul/extras/perl-Sysadm-Install/perl-Sysadm-Install...