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=533302 Dominic Hopf <dmaphy@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #3 from Dominic Hopf <dmaphy@xxxxxxxxxxxxxxxxx> 2009-11-13 08:28:43 EDT --- Just a small addition to the %files-section: %files %defattr(-,root,root,-) %doc CHANGES.txt Contributors.txt LICENSE.txt README.txt doc/manual.pdf %{_bindir}/%{name} %{python_sitelib}/%{name}* %{_mandir}/man1/* The %{name} macros are important to make sure no files other than the ones delivered with your package are affected. (See check point "Package does not own files or directories owned by other packages." in the list above.). The %{python_sitelib}-line actually was okay, but the %{name} macro is a bit more elegant. Before you import your Specfile to CVS, please add those macros. Anything else is okay now. The package is approved. -- 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