> Thanks a lot, overall it's looking very nice, I have a couple of comments
> 15 +if test "x$enable_fts" = "x" ; then
> 16 + enable_fts=yes
> 17 +fi
> Looks very odd, please use the optional parameters of AC_ARG_ENABLE instead.
I had problems with this optional parameter before, but it works now.
> 62 + if (siin != null) return;
> Not needed.
> 64 + "org.gnome.zeitgeist.SimpleIndexer",
> Let's use INDEXER_NAME here as well.
> 90 + siin_connection_failed = true; //TODO: Is this
> Yes, it should be there.
ok these small changes are also done.
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.
Mailing list: https://launchpad.net/~zeitgeist
Post to : firstname.lastname@example.org
Unsubscribe : https://launchpad.net/~zeitgeist
More help : https://help.launchpad.net/ListHelp