← Back to team overview

zeitgeist team mailing list archive

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