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=542436 Rahul Sundaram <metherid@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |metherid@xxxxxxxxx --- Comment #12 from Rahul Sundaram <metherid@xxxxxxxxx> 2011-01-31 11:07:33 EST --- You should probably open up a new review request instead of reopening a old new since the reports counting reviews processes will get confused otherwise and you shouldn't assign the review request to yourself. Some preliminary notes * Should document why you are using the git version instead of the release in the spec file * Should document the patch and their upstream status * No need to define the buildroot or have a clean section unless you are branching for EPEL -- 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