zeitgeist team mailing list archive
-
zeitgeist team
-
Mailing list archive
-
Message #04775
Re: [Merge] lp:~zeitgeist/zeitgeist/fts++ into lp:zeitgeist
I like the last one :)
On Fri, Feb 10, 2012 at 12:30 PM, Siegfried Gevatter <rainct@xxxxxxxxxx>wrote:
> Review: Approve
>
> OK, merging it, but there's some outstanding stuff:
>
> * Important: the TableLookup in FTS can currently explode. The new schema
> version needs to add AUTOINCREMENT to the `id' row of all tables in
> TableLookup. We should do this before releasing a new tarball.
>
> * TableLookup.get_value: please prepare the query in the constructor.
>
> * Configure isn't checking for xapian being there
>
> * Add a flag to disable FTS? (keeping the Xapian dependency avoidable)
> --
> https://code.launchpad.net/~zeitgeist/zeitgeist/fts++/+merge/92022
> You are subscribed to branch lp:zeitgeist.
>
--
https://code.launchpad.net/~zeitgeist/zeitgeist/fts++/+merge/92022
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.
References