D7474: Remove hard X11 runtime dependency from DBusInterface

Martin Flöser noreply at phabricator.kde.org
Wed Aug 23 09:17:48 UTC 2017


graesslin created this revision.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  KWin's DBusInterface announces the service name on an X11 root window
  property. For this it requires a runtime dependency on X11. If we would
  try to start kwin_wayland without XWayland support this would result in
  a crash.
  
  This change turns the dependency into an optional one by checking
  whether the KWin::Application has support for X11 and registers a
  connection to announce the property as soon as a connection is
  established. Thus it even supports the restart of XWayland now.

TEST PLAN
  Compiles

REPOSITORY
  R108 KWin

BRANCH
  dbus-interface-no-x11-dependency

REVISION DETAIL
  https://phabricator.kde.org/D7474

AFFECTED FILES
  dbusinterface.cpp

To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170823/13d1c86f/attachment-0001.html>


More information about the Plasma-devel mailing list