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=796183 --- Comment #2 from Bohuslav "Slavek" Kabrda <bkabrda@xxxxxxxxxx> 2012-02-22 08:58:39 EST --- - URL should probably point to [1], which contains some information about the project, not the directory with source files/tars/etc... - You should use "install -p" to preserve timestamps, see [2]. - You cannot own %{python_sitelib} - that would imply that you own the whole directory with any files _even from other packages_ (and after uninstallation of your package, it would remove this directory with everything inside it). Rather than that, use %{python_sitelib}/*, which will cause your package to own all the files that are under %{buildroot}%{python_sitelib}. - Otherwise the package looks good and works, so when you fix these problems, it can be approved. [1] http://psss.fedorapeople.org/python-nitrate/python-nitrate.html [2] https://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review