Re: [Tracker] Tracker GConf should use DBus session bus



On 24/08/12 09:48, JÃrg Billeter wrote:
On Fri, 2012-08-24 at 17:21 +0900, Richard Dale wrote:
On Fri, Aug 24, 2012 at 4:58 PM, JÃrg Billeter <j bitron ch> wrote:
On Fri, 2012-08-24 at 09:45 +0200, Philip Van Hoof wrote:
I think we made it this way because the key that we needed on Harmattan
was in the system bus.

So if we would change it we should probably add an #ifdef for harmattan
so that on this platform it remains the way it is now.

The whole file is exclusively for Meego. On other systems, locale
settings are not stored in GConf and we use the environment variables.
I am developing on a Meego derived platform, although not Harmattan.

On Harmattan it was definitely correct to use the system bus. If you can
verify that upstream Meego does not use the system bus for GConf, I
think we should simply change it given that Harmattan will never use
tracker master.

Thanks JÃrg, Philip,

While we could change master here, I actually wonder where we're using this Harmattan code?

Is it on MeeGo?
Is it for general purpose (i.e. someone wants to use gconf with dbus)?

Depending on these answers, we can either:

a) fix in master and rename as Philip suggests
b) create a new .c file for that target

I am fine with either as long as we don't disrupt anyone using Tracker on existing platforms. Richard, patches/branches welcome ;)

--
Regards,
Martyn

Founder and CEO of Lanedo GmbH.



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]