GNOME Bugzilla – Bug 547478
libgweather shouldn't keep a hard dependency on gconf
Last modified: 2011-04-21 14:02:57 UTC
libgweather is a great initiative but in order to really succeed I think it should be usable regardless of the desktop environment you're using. However, it depends on GConf which is tightly linked to Gnome. I would be interested in porting/adapting/rewriting the weather plugin/applet of Xfce using libgweather but unfortunately depending on GConf is a showstopper. Please don't take this report the wrong way. I'm not trying to tell you what to do, just suggesting a way to improve the way libgweather can be used and also to "broaden its audience". PS: on the long run I think the libxml dependency can also be removed by using a GLib-based parser.
The current usage of GConf is broken anyway; GWeatherGConf only stores gweather-applet's preferences. The clock applet makes its own completely separate GConf calls. So we should just move GWeatherGConf back to gweather-applet. > PS: in the long run I think the libxml dependency can also be removed by using > a GLib-based parser. Yes... it's unlikely that any of the maintainers would bother to do that, but if you wanted to submit a patch we'd look at it. One possible problem is that reading in Locations.xml is already noticeably slow, and GMarkup is probably less optimized than libxml (?), so that might make it even slower. If we're going to redo the parsing again, a better fix might be to have the build process convert the XML format into something directly-mmappable, so we didn't have to parse it at run time.
I wonder what's the point of removing the libxml dependency: it's a library that is widely used and will be most likely (as in 99%) be installed anyway. Agree for gconf, although we might want to have a "Use desktop-wide setting" option which would use the configuration stored in gconf for GNOME... So. Who wants to move some code to the weather applet? :-)
This bug is a duplicate of 646854, which is newer but has a patch (and is blocking the GNOME Goal).
*** This bug has been marked as a duplicate of bug 646854 ***