Review Request: Add option to KGlobalConfig::activate() to skip connection to dbus

George Kiagiadakis kiagiadakis.george at gmail.com
Thu Aug 19 16:48:48 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5072/
-----------------------------------------------------------

(Updated 2010-08-19 15:48:48.210762)


Review request for kdelibs, David Faure and Oswald Buddenhagen.


Changes
-------

Updated the diff to address David's comments.


Summary
-------

This patch modifies KGlobalConfig so that it is possible for applications to tell it not to connect to dbus and listen for configuration changes from other applications. This is useful for kdm, because if QDBusConnection::sessionBus() is called while there is no session bus running, a new session bus is automatically created, which makes no sense for kdm.


Diffs (updated)
-----

  trunk/KDE/kdelibs/kdeui/kernel/kglobalsettings.h 1165324 
  trunk/KDE/kdelibs/kdeui/kernel/kglobalsettings.cpp 1165324 

Diff: http://reviewboard.kde.org/r/5072/diff


Testing
-------

Tested with kdm after modifying its call to activate() to use the DontListenForChanges flag. kdm no longer starts a dbus session.


Thanks,

George

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100819/fc91583c/attachment.htm>


More information about the kde-core-devel mailing list