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: pyxapian - Xapian python bindings https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244357 jeff@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |jeff@xxxxxxxxxx Flag| |fedora-review? ------- Additional Comments From jeff@xxxxxxxxxx 2007-06-15 09:16 EST ------- Just some quick notes: 1. the following lines are no longer required: %define pyver %(%{__python} -c "import sys ; print sys.version[:3]") Requires: python-abi = %{pyver} RPM will pick up the appropriate python-abi requirements automatically 2. The files section can be simplified to: %{python_sitelib} 3. It would be nice if upstream would include a copy of the GPL in the tarball and make the appropriate changes to the PKG-INFO file. -- 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