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=458279 Till Maas <opensource@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |opensource@xxxxxxxxx --- Comment #1 from Till Maas <opensource@xxxxxxxxx> 2009-04-18 11:12:28 EDT --- Just two remarks I spoted: The Source0: URL for SF.net is downloads.sourceforge... not download.sourceforge..., there is an "s" missing. Instead of %define, %global should be used in the first line of the spec. I will try to make a full review later this day, so there is no need to update the spec yet for these two issues. -- 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