Bug 546704 - Review Request: python-whoosh - Fast, pure-Python full text indexing, search, and spell checking library
Summary: Review Request: python-whoosh - Fast, pure-Python full text indexing, search,...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Thomas Spura
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-12-11 18:19 UTC by Allisson Azevedo
Modified: 2011-10-01 11:15 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-10-01 11:15:49 UTC
Type: ---
Embargoed:
tomspur: fedora-review?


Attachments (Terms of Use)

Description Allisson Azevedo 2009-12-11 18:19:06 UTC
Spec URL: http://allisson.fedorapeople.org/packages/python-whoosh/python-whoosh.spec

SRPM URL: http://allisson.fedorapeople.org/packages/python-whoosh/python-whoosh-0.3.9-1.fc12.src.rpm

Description: Whoosh is a fast, featureful full-text indexing and searching library 
implemented in pure Python.

Whoosh was created and is maintained by MattChaput. It was originally created 
for use in the online help system of Side Effects Software's 3D animation 
software Houdini. Side Effects Software Inc. graciously agreed to open-source 
the code.

Some of Whoosh's features include:

Pythonic API.
Pure-Python. No compilation or binary packages needed, no mysterious crashes.
Fielded indexing and search.
Fast indexing and retrieval -- faster than any other pure-Python, scoring, 
full-text search solution I know of.
Pluggable scoring algorithm (including BM25F), text analysis, storage, posting 
format, etc.
Powerful query language parsed by pyparsing.
Pure Python spell-checker (as far as I know, the only one).
Whoosh might be useful in the following circumstances:

Anywhere a pure-Python solution is desirable to avoid having to build/compile 
native libraries (or force users to build/compile them).
As a research platform (at least for programmers that find Python easier to 
read and work with than Java ;)
When an easy-to-use Pythonic interface is more important to you than raw speed.

Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1870135

Comment 1 Thomas Spura 2009-12-17 12:24:41 UTC
Review:

Good:
- name ok
- using macros
- buildroot ok
- rpmlint clean
- %clean section is there
- permissions ok
- %files section ok
- arch ok
- no .la files
- no libs
- no desktop file needed
- builds in koji


NEEDSWORK:

- description is bad: If you do a 'yum search Java' you'll get this package, which has nothing to do with java...

in PKG-INFO is:
        Whoosh is a fast, pure-Python indexing and search library. Programmers

        can use it to easily add search functionality to their applications and

        websites. Because Whoosh is pure Python, you don't have to compile or

        install a binary support library and/or make Python work with a JVM, yet

        Whoosh is still very fast at indexing and searching. Every part of how

        Whoosh works can be extended or replaced to meet your needs exactly.

How about that?

- BR wrong: please require only python-setuptools and not python-setuptools-devel

- You should query upstream to include a license.txt or COPYING file. It's referenced in PKG-INFO, but nowhere to find.

Comment 2 Allisson Azevedo 2009-12-18 12:38:12 UTC
Update package:

Spec URL:
http://allisson.fedorapeople.org/packages/python-whoosh/python-whoosh.spec

SRPM URL:
http://allisson.fedorapeople.org/packages/python-whoosh/python-whoosh-0.3.9-2.fc12.src.rpm

Changelog
- Fix BR.
- Copy LICENSE.txt from svn to include in rpm.
- Fix description

Comment 3 Thomas Spura 2009-12-18 14:19:01 UTC
See: https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text

Please delete the LICENSE.txt file again. Upstream *should* include it, if you email them and they say 'no' - that's ok. You only need to mail them, and that's it.

Comment 4 Thomas Spura 2010-04-24 19:20:59 UTC
There is a new version 0.3.18, could you please update?

The URL is now wrong, because upstream uses now bitbucket:
http://bitbucket.org/mchaput/whoosh/wiki/Home

There should not arise a new problem for this to get approved...

Comment 5 Thomas Spura 2010-05-29 13:50:46 UTC
Ping, are you still interessted in packaging this?

Comment 6 Allisson Azevedo 2010-05-29 14:14:26 UTC
Hi, i'll make a update soon.

And yes, i interessted in packaging this.

Regards,

Allisson.

Comment 7 Thomas Spura 2010-10-22 12:40:09 UTC
Another ping...

Comment 8 Silas Sewell 2010-12-05 02:23:15 UTC
I've created a separate spec (didn't see this review until I was about to submit a review request), it looks like this review is probably stalled.

Allisson please respond to this ticket within a week otherwise I'm going to close this ticket as dead and submit a new review.

http://fedoraproject.org/wiki/Extras/Policy/StalledReviews#Submitter_not_responding

If you're still interested in maintaining this package you're welcome to grab my spec (you're missing at least the licensing stuff and check section).

SPEC: https://github.com/silas/rpms/raw/master/python-whoosh/python-whoosh.spec

Comment 9 Thomas Spura 2011-10-01 11:15:49 UTC
@Silas: You should open your review request.

Closing.


Note You need to log in before you can comment on or make changes to this bug.