[Bug 785416] Review Request: python-xappy - A Python module providing an easy-to-use layer on top of the Xapian search engine

bugzilla at redhat.com bugzilla at redhat.com
Tue Mar 13 07:16:17 UTC 2012


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=785416

--- Comment #2 from Haïkel Guémar <karlthered at gmail.com> 2012-03-13 03:16:15 EDT ---
Few fixes before formal review starts:
* there's a typo in the license
* Summary is way too long (e.g: A python high-level wrapper to Xapian search
engine)
* missing requires: xapian-bindings-python 
* you should remove the shebang line from the files in xappy.cachemanager, not
required for executing modules as scripts.
* if you generated the tarball manually, please add a comment explaining how it
was generated and add the license file (upstream latest tarball does at least)
* if you plan not to support EPEL5, please remove the buildroot tag, the
buildroot manual cleaning and the defattr macros.

-- 
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.


More information about the package-review mailing list