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=490576 Jussi Lehtola <jussi.lehtola@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |jussi.lehtola@xxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |jussi.lehtola@xxxxxx Flag| |fedora-review? --- Comment #1 from Jussi Lehtola <jussi.lehtola@xxxxxx> 2009-03-17 19:32:28 EDT --- I can do the review on this one. A few quick notes: - change all of the %define:s to %global:s since the behaviour is going to change shortly (defines are only meant to be local definitions according to RPM specifications). - Why do you turn off building of the debug package? Add a comment on this to the spec file. - You need to preserve time stamps while copying: change all 'cp' to 'cp -p'. - Man pages should not be in %doc; remove the %{doc} prefix from the files section for the man pages. Also, you might want to shorten the whole thing to just %{_mandir}/man1/*.1.gz -- 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