Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 281095 - dev-python/Whoosh (New package)
Summary: dev-python/Whoosh (New package)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Anthony Basile
URL: http://whoosh.ca/
Whiteboard: sunrise-overlay
Keywords: EBUILD, InOverlay
Depends on:
Blocks: 281096
  Show dependency tree
 
Reported: 2009-08-11 12:02 UTC by Michał Górny
Modified: 2012-05-22 12:51 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Fresh ebuild (Whoosh-0.2.6.ebuild,386 bytes, text/plain)
2009-08-13 12:52 UTC, Michał Górny
Details
whoosh-2.4.0.ebuild (whoosh-2.4.0.ebuild,812 bytes, text/plain)
2012-05-06 22:31 UTC, Johan Bergström
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2009-08-11 12:02:46 UTC
Whoosh is a full-text indexing and search engine written entirely in Python. It can be used as a django-haystack backend for example.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2009-08-13 12:52:05 UTC
Created attachment 201124 [details]
Fresh ebuild

Example ebuild for current version of Whoosh. Same ebuild works for 0.1.24 (which is required by django-haystack).
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2009-08-20 15:48:29 UTC
Ebuilds are now in Sunrise overlay, you can review them using the following URL:
  http://overlays.gentoo.org/proj/sunrise/browser/reviewed/dev-python/Whoosh
Comment 3 Johan Bergström 2012-05-06 22:31:22 UTC
Created attachment 311047 [details]
whoosh-2.4.0.ebuild

Updated ebuild
Comment 4 Johan Bergström 2012-05-06 22:32:36 UTC
Lets bring this into the tree. CC:ing blueness since he's interested, python for relevance and proxy if there's anyone interested in co-maintaining this with me \o/
Comment 5 Anthony Basile gentoo-dev 2012-05-06 23:05:10 UTC
I'll take care of it.  Ping in in about 1 week.  Exam period = hell!
Comment 6 Anthony Basile gentoo-dev 2012-05-21 11:38:29 UTC
I added Whoosh-2.3.2.ebuild from the sunrise overlay.
Comment 7 Johan Bergström 2012-05-21 12:20:50 UTC
Hey,
thanks for spending time on this!

Any specific reason you didn't pick the updated ebuild I attached to this bug?
Comment 8 Anthony Basile gentoo-dev 2012-05-21 13:00:05 UTC
(In reply to comment #7)
> Hey,
> thanks for spending time on this!
> 
> Any specific reason you didn't pick the updated ebuild I attached to this
> bug?

No, here's what happened: since the sunrise ebuild was mature, I put it on the tree first.  I then went to put the attached one on the tree and noticed you had changed from upper to lower case.  I did not continue until I spoke with you first.  I've tested both and they're good to go.

If we're okay with the upper case name "Whoosh" then I'll make the minor changes the attached ebuild to accomodate that.  If not, then we have to migrate.  Sorry about the mixup, but I just didn't notice the upper <-> lower switch.

Advantages to upper: 1) follows upstream naming scheme. 2) slightly easier to maintain.  Disadvantage: 1) most, but not all, packages in the tree use lower case only.

Advantages to lower: 1) packages in tree tend to be lower case.  Disadvantage: 1) now we have to migrate :(

Personal preference: I always try to follow upstream whenever possible to respect their naming convention and there is no breakage with upper <-> lower case, just a convention.

However, we can switch if you want.
Comment 9 Anthony Basile gentoo-dev 2012-05-21 13:32:15 UTC
> However, we can switch if you want.

Actually I spoke with some other devs and their general consensus is "sucks to type" but there's no necessary reason, see dev-perl.  So I'll wait for an ack and then make the change.
Comment 10 Johan Bergström 2012-05-21 21:05:08 UTC
(In reply to comment #9)
> > However, we can switch if you want.
> 
> Actually I spoke with some other devs and their general consensus is "sucks
> to type" but there's no necessary reason, see dev-perl.  So I'll wait for an
> ack and then make the change.

I'm fine with whatever, really. My optimal route would be to get an "definitive" answer regarding this - since it feels "wrong" to uppercase, but I can't motivate why (other than something like the reply you got). One argument in this specific case could be that the python modname is actually called "whoosh" (import whoosh).

I'm more interested in getting the latest version in tree as well as adding python 3 support (remove restrict).
Comment 11 Anthony Basile gentoo-dev 2012-05-22 12:51:39 UTC
(In reply to comment #10)
> (In reply to comment #9)
> > > However, we can switch if you want.
> > 
> > Actually I spoke with some other devs and their general consensus is "sucks
> > to type" but there's no necessary reason, see dev-perl.  So I'll wait for an
> > ack and then make the change.
> 
> I'm fine with whatever, really. My optimal route would be to get an
> "definitive" answer regarding this - since it feels "wrong" to uppercase,
> but I can't motivate why (other than something like the reply you got). One
> argument in this specific case could be that the python modname is actually
> called "whoosh" (import whoosh).
> 
> I'm more interested in getting the latest version in tree as well as adding
> python 3 support (remove restrict).

Okay, I think we're good now:

1) dev-python/Whoosh is now renamed to dev-python/whoosh.

2) The ebuild for Whoosh-2.3.2 is gone.  I did not move Whoosh-2.3.2.ebuild to whoosh-2.3.2.ebuild

3) whoosh-2.4.0.ebuild is on the tree

Take a look Johan, and let me know if this works for you.  If we need/want 2.3.2 back, its an easy write, I'll put it back.  But it looks like 2.4.0 is better because it is python-3 compat.