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@xxxxxxxxxxxx> 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.spec http://www.city-fan.org/~paul/extras/perl-Sysadm-Install/perl-Sysadm-Install-0.27-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review