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=772751 --- Comment #2 from Pete Zaitcev <zaitcev@xxxxxxxxxx> 2012-01-17 21:41:38 EST --- I reviewed Spenser's informal review and fixed up the submission. The failed check points are fully addressed, but with non-mandatory things, I had to maneuver a little: - the spelling fail is due to "rfc.sh", so left it untouched - not sure if we want to run dos2unix on LICENSE ... - %clean is left in place for EPEL For some reason it was a complete news to me that %doc installs, sorry. RPMs are updated, here: http://people.redhat.com/zaitcev/tmp/git-review-1.9-3.fc16.src.rpm http://people.redhat.com/zaitcev/tmp/git-review-1.9-3.fc16.spec -- 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