daevaorn / djapian

High level Xapian integration for Django
Other
6 stars 3 forks source link

Xapian 1.2 support #113

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm aiming to get Xapian 1.2 into the next Debian stable release, so I've 
been checking all the Debian packages which depend on Xapian.

Xapian 1.2.0 drops support for some deprecated features, and djapian seems 
to use a few of these:

http://trac.xapian.org/browser/trunk/xapian-core/docs/deprecation.rst#id5

I'll attach a patch, but it is untested as I don't actually use djapian and 
don't know enough about it to write a test.  There seem to be some tests in 
the djapian source, but I didn't see how to run them.

Original issue reported on code.google.com by ojwbe...@gmail.com on 30 Apr 2010 at 10:02

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by daevaorn on 30 Apr 2010 at 10:31

GoogleCodeExporter commented 9 years ago
BTW, I forgot to mention that with the patch the code should work with Xapian 
1.0.x 
and 1.2.x.

Original comment by ojwbe...@gmail.com on 30 Apr 2010 at 2:05

GoogleCodeExporter commented 9 years ago
We've just had someone on #xapian on IRC who had hit this incompatibility.

I think you probably want to apply this patch before 2.5 if you don't want to
frustrate many potential new djapian users.

Original comment by ojwbe...@gmail.com on 23 May 2010 at 12:52

GoogleCodeExporter commented 9 years ago
The patch has been committed in r365. It would be better to check for other 
obsoleted things before closing the issue, though.

Original comment by esizi...@gmail.com on 21 Jun 2010 at 4:05

GoogleCodeExporter commented 9 years ago

Original comment by daevaorn on 21 Jun 2010 at 9:04

GoogleCodeExporter commented 9 years ago
Closed. Thanks Olly!

Our new release will be soon.

Original comment by daevaorn on 23 Jun 2010 at 10:24

GoogleCodeExporter commented 9 years ago
In r366 additional fixes were made: get_collapse_count() and get_collapse_key() 
methods of MSetItem class are obsoleted by collapse_count and collapse_key 
properties since Xapian 1.0 release and removed in Xapian 1.2

Original comment by esizi...@gmail.com on 30 Jun 2010 at 7:29

GoogleCodeExporter commented 9 years ago
Issue 118 has been merged into this issue.

Original comment by daevaorn on 21 Jul 2010 at 7:41