Re: [Tracker] Fixing private headers included from public headers

[Jürg Billeter <j bitron ch>]

On Thu, 2010-02-11 at 10:07 +0100, Martyn Russell wrote:
> On 11/02/10 01:34, Martyn Russell wrote:
> > It looks like we need to make these publicly installable for libraries
> > like libtracker-extract, etc:
> >
> > tracker-dbus.h (for libtracker-miner)
> > tracker-statement-list.h (for libtracker-extract)
> > tracker-sparql-builder.h (referenced from tracker-statement-list.h)
>
> After further reflection, I realised that tracker-dbus.h is not included 
> by any headers for libtracker-miner. So we could simply keep that in 
> libtracker-common and put the other two headers into libtracker-client. 
> The reason this would work is that libtracker-extract depends on 
> libtracker-client for linking anyway and it seems a more logical place 
> to put these modules.

Moving tracker-sparql-builder.* to libtracker-client sounds good to me.
However, tracker-statement-list.* mainly contains obsolete functions, we
should avoid exposing this as part of the public API. The
tracker_sparql_builder_object_unvalidated function from that file should
be public but the rest should be private or better removed.

JÃrg