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=511938 Fabian Affolter <fabian@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fabian@xxxxxxxxxxxxxxxxx --- Comment #1 from Fabian Affolter <fabian@xxxxxxxxxxxxxxxxx> 2009-07-16 04:30:40 EDT --- Some quick comments on this spec file too. - You are mixing '$RPM_BUILD_ROOT' and '%{buildroot}' (as in #511941) - 'Source0' should point to the upstream location of the source tarball. https://fedoraproject.org/wiki/Packaging/SourceURL If this is not possible add a comment about how the checkout from the VCS was done. - 'Requires: pkgconfig' is missing in the -devel package https://fedoraproject.org/wiki/Packaging:Guidelines#Pkgconfig_Files -- 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