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=731219 Peter Gordon <peter@xxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Alias| |pipebench --- Comment #3 from Peter Gordon <peter@xxxxxxxxxxxxxxxx> 2011-08-19 17:13:24 EDT --- Thank you for your feedback, Nathan. With regards to using the sed invocation instead of the patch you provided, I appreciate the effort but I think it is unncessary. It's just a few search-and-replace changes; and I feel that using a patch instead would be adding another layer of complexity without just cause. Also, I use the glob for the man pages as a convenience to myself, mostly force of habit, as I'm accustomed to larger packages where using the "man?/" pattern catches the man pages irrespective of the section they're installed into. It's a tiny bit of future-proofing that I think makes things a little nicer aesthetically. If you'd like me to explicitly list each section directory, I'd be happy to adjust that in the spec file. :) -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review