Re: [Tracker] Anonymous File Nodes branch review

2010-02-03 Thread Philip Van Hoof
On Wed, 2010-02-03 at 12:08 +, Martyn Russell wrote: Hi all, Carlos and Philip recently worked on the anonymous-file-nodes branch to fix a regression with the way resources are stored in the database which was changed recently. This is the review of that branch:

Re: [Tracker] Anonymous File Nodes branch review

2010-02-03 Thread Carlos Garnacho
Hi! On miƩ, 2010-02-03 at 12:08 +, Martyn Russell wrote: Hi all, Carlos and Philip recently worked on the anonymous-file-nodes branch to fix a regression with the way resources are stored in the database which was changed recently. This is the review of that branch:

Re: [Tracker] Anonymous File Nodes branch review

2010-02-03 Thread Martyn Russell
On 03/02/10 12:51, Philip Van Hoof wrote: On Wed, 2010-02-03 at 12:08 +, Martyn Russell wrote: - tracker_sparql_builder_object_iri (sparql, uri); + tracker_sparql_builder_object (sparql, _:foo); Looks right to me Just to repeat what I said on IRC: Technically there is

Re: [Tracker] Anonymous File Nodes branch review

2010-02-03 Thread Martyn Russell
On 03/02/10 13:05, Carlos Garnacho wrote: 1. We should use G_DIR_SEPARATOR_S here instead: + slash_uri = g_strconcat (source_uri, /, NULL); Hmm, we're building an uri here, I don't think we want uris with '\' if tracker is ported (again?) to Windows :) This is a good point :P 2.

Re: [Tracker] Anonymous File Nodes branch review

2010-02-03 Thread Martyn Russell
On 03/02/10 12:08, Martyn Russell wrote: Hi all, Carlos and Philip recently worked on the anonymous-file-nodes branch to fix a regression with the way resources are stored in the database which was changed recently. This is the review of that branch:

Re: [Tracker] Thumbnail branch review

2010-02-03 Thread Martyn Russell
On 01/02/10 13:31, Martyn Russell wrote: Hi all, Philip recently re-added some thumbnail work in a branch, this is the review of that branch: http://git.gnome.org/browse/tracker/log/?h=thumbnails This has been merged now. -- Regards, Martyn ___

[Tracker] upcoming 0.6.96 release

2010-02-03 Thread Michael Biebl
Hi Tracker devs, hi Martyn, did some tests on the current tracker-0.6 branch in preparation of the 0.6.96 release. I tested in a clean chroot, where no previous installation of tracker is around I use LDFLAGS = -Wl,--as-needed -Wl,-Bsymbolic and the following configure flags