GNOME Bugzilla – Bug 622880
Migrate from dbus-glib to glib's GDBus
Last modified: 2014-08-20 20:23:41 UTC
For GLib 2.25.5 GDBus D-Bus support was merged, providing an API to replace dbus-glib. See http://library.gnome.org/devel/gio/unstable/gdbus.html and http://library.gnome.org/devel/gio/unstable/ch28.html . According to a quick grep this module seems to use dbus-glib: ./gnome-screensaver/configure.ac: dbus-glib-1 >= $DBUS_REQUIRED_VERSION ./gnome-screensaver/configure.ac: dbus-glib-1 >= $DBUS_REQUIRED_VERSION)
*** Bug 644279 has been marked as a duplicate of this bug. ***
*** Bug 644280 has been marked as a duplicate of this bug. ***
*** Bug 644281 has been marked as a duplicate of this bug. ***
*** Bug 644282 has been marked as a duplicate of this bug. ***
Saleem, can you add all of your patches here please?
Sure, I just need to dig them up, and make sure that they still apply to master.
Created attachment 234165 [details] [review] 0001-saver-port-watcher-to-GDBus.patch
Created attachment 234166 [details] [review] 0002-saver-port-listener-to-GDBus.patch
Created attachment 234167 [details] [review] 0003-build-no-longer-link-against-dbus-glib.patch
Created attachment 234168 [details] [review] 0004-build-remove-unused-DBUS-references.patch
These are rebased to current master. I've not really tested them much beyond a compile cycle after the rebasing. Though, fortunately, the changes weren't too extensive. IIRC, everything except the message bit I had tested previously (not that I really consider that valid given the patchset was rebased).
The screen lock is now implemented directly in gnome-shell, not in gnome-screensaver (as it was with older version of GNOME 3 and GNOME 2.x). If the reported problems persists in recent versions of GNOME, please file a new bug against gnome-shell with the "lock-screen" component selected. If your installation is an older version of GNOME that cannot be upgraded (such as an enterprise version), please use the appropriate support mechanism from your vendor, or your distribution instead.