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=525764 Rahul Sundaram <sundaram@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #2 from Rahul Sundaram <sundaram@xxxxxxxxxx> 2009-09-25 14:50:00 EDT --- APPROVED The only comment I have to add is to consider removing the buildroot definition and removal of it in the install section unless to reduce spec clutter unless you are branching for EPEL as well. -- 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