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@xxxxxxxxx> 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review