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=743612 Michael Scherer <misc@xxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |misc@xxxxxxxx --- Comment #2 from Michael Scherer <misc@xxxxxxxx> 2012-01-20 12:24:29 EST --- A few remark : - patchs are not commented, and I think they should explain why they are here, and if pushed upstream or not - Buildroot: should be removed - I think %defattr is deprecated as well - %dist is missing from release I am not sure that the authors name and the setup instruction belong to %description. -- 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