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=503519 Tom "spot" Callaway <tcallawa@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|182235(FE-Legal) | AssignedTo|nobody@xxxxxxxxxxxxxxxxx |tcallawa@xxxxxxxxxx Flag| |fedora-review? --- Comment #5 from Tom "spot" Callaway <tcallawa@xxxxxxxxxx> 2009-06-04 09:10:26 EDT --- Trademark stuff looks good now, thanks. A few other issues that need to be resolved: * The License: should be GPLv3+ (look at the code headers). * You're also not building it with %{optflags}. (passing CXXFLAGS="%{optflags}" to make should do the trick) * You're also using the old icon cache scriptlets, see: https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache * You do not need to manually gzip the manpage, just install it uncompressed into the proper mandir and rpm will gzip it properly for you. I think that fixing those items should be enough for me to finish the review. Lifting FE-Legal. -- 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