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=526651 Jon Ciesla <limb@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |limb@xxxxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |limb@xxxxxxxxxxxx Flag| |fedora-review? --- Comment #33 from Jon Ciesla <limb@xxxxxxxxxxxx> 2009-11-04 10:59:11 EDT --- Hmm, a few issues on a full review. One, they didn't update the GPL licensing text in all places, but that's OK since the GPLv2 stuff says GPLv2+. Given that, might want to change license tage to GPLv3+. Source URL should be Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz, modified for the project name. Why is the URL tag pointing to the SF project, and not the SF web page? Mock build and BRs are OK. I've also read this entire review and considered the issues raised and their solutions. I think once the various tags above are fixed, I could approve. -- 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