Review Request 122339: Add KWorkSpace::isShuttingDown()

Kai Uwe Broulik kde at privat.broulik.de
Mon Feb 9 19:10:19 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122339/
-----------------------------------------------------------

(Updated Feb. 9, 2015, 7:10 nachm.)


Review request for Plasma and Martin Gräßlin.


Changes
-------

Remove X11 ifdef, we should perhaps guard against the availability of QtDBus, not that Plasma or any other workspace part would build without it though.


Bugs: 189456
    https://bugs.kde.org/show_bug.cgi?id=189456


Repository: plasma-workspace


Description
-------

Since KSMServer does the session teardown, we can't just rely on Logind's PreparingForShutdown property.

This patch will allow for instance PowerDevil to not suspend the session when it's currently shutting down, otherwise you'll have your system suspend and continue shutting down after resuming.


Diffs (updated)
-----

  ksmserver/org.kde.KSMServerInterface.xml 9dad130 
  ksmserver/server.h 20b5faa 
  libkworkspace/kworkspace.h 816f8f1 
  libkworkspace/kworkspace.cpp ac979bd 

Diff: https://git.reviewboard.kde.org/r/122339/diff/


Testing
-------

Works, adjusted PowerDevil and the system did not suspend during shutdown.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150209/22e5ddda/attachment.html>


More information about the Plasma-devel mailing list