Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pyxmms https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195393 ------- Additional Comments From paul@xxxxxxxxxxxx 2006-06-20 03:35 EST ------- A couple of points: 1. Simply adding "%ghost /%{python_sitelib}/xmms/*.pyo" is not sufficient since the .pyo files are already included in the package courtesy of the previous line "/%{python_sitearch}/xmms/". The standard idiom for this is: %dir %{python_sitearch}/xmms/ %{python_sitearch}/xmms/*.py %{python_sitearch}/xmms/*.pyc %ghost %{python_sitearch}/xmms/*.pyo In this case it will also be necessary to have: %{python_sitearch}/xmms/*.so Note that no leading slash is necessary for %{python_sitearch} as the macro expansion includes one already. 2. The version of the package imported into cvs does not have the spec file from Comment #2. Please be more careful about what you import into cvs (and build?). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review