Re: ‘class Xapian::Database’ has no member named ‘close’

Subject: Re: ‘class Xapian::Database’ has no member named ‘close’

Date: Tue, 14 Aug 2012 14:26:13 +0200

To: David Bremner, notmuch@notmuchmail.org

Cc:

From: Michal Nazarewicz


> Michal Nazarewicz <mina86@mina86.com> writes:
>> $ dpkg -l |grep xapian
>> ii  apt-xapian-index  0.25ubuntu2      maintenance tools for a Xapian index of Debi
>> ii  libxapian-dev     1.0.18-1         Development files for Xapian search engine l
>> ii  libxapian15       1.0.18-1         Search engine library

David Bremner <david@tethera.net> writes:
> This might be a bug with the install documentation; it could be we
> really need a later xapian to build.

Yeah, it seems that INSTALL claims “Notmuch will work best with Xapian
1.0.18 (or later) or Xapian 1.1.4 (or later)”, which doesn't seem to be
the case.

On top of that, configure does not seem to even check the version
number.

> From memory, maybe 1.0.20 has the "close" method. 1.0.18 is pretty
> old; Debian squeeze has 1.2.3, and that must be almost 3 years old
> now.

Is close() really required though?  The comment justifies it saying
that:

	Many Xapian objects hold references to the database, so merely
	deleting the database may not suffice to close it.  Thus, we
	explicitly close it here.

but wouldn't the database get closed when the last reference gets
deleted anyway?

-- 
Best regards,                                         _     _
.o. | Liege of Serenely Enlightened Majesty of      o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz    (o o)
ooo +----<email/xmpp: mpn@google.com>--------------ooO--(_)--Ooo--

Thread: