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=475897 Fabian Affolter <fabian@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |fabian@xxxxxxxxxxxxxxxxx Flag| |fedora-review? --- Comment #2 from Fabian Affolter <fabian@xxxxxxxxxxxxxxxxx> 2009-01-12 05:26:13 EDT --- Some comment before I do a full review. - '-n %{name}-%{version}' is not needed in %setup because this is the default. - ChangeLog is missing in %doc - I personally like one line per BR ;-) -- 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