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