Review Request 122339: Add KWorkSpace::isShuttingDown()

[Kai Uwe Broulik]

> On Feb. 2, 2015, 7:45 vorm., Martin Gräßlin wrote:
> > is that for the case that no logind is used? Or should ksmserver better integrate with logind?

KSMServer handles the session teardown and only after that does Logind get involved which means PreparingForShutdown is false until the Plasma session quits which is why I need this method.


- Kai Uwe


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


On Jan. 31, 2015, 4:02 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122339/
> -----------------------------------------------------------
> 
> (Updated Jan. 31, 2015, 4:02 nachm.)
> 
> 
> Review request for Plasma and Martin Gräßlin.
> 
> 
> 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
> -----
> 
>   ksmserver/shutdown.cpp e6bc708 
>   libkworkspace/kworkspace.h 816f8f1 
>   libkworkspace/kworkspace.cpp ac979bd 
>   ksmserver/org.kde.KSMServerInterface.xml 9dad130 
>   ksmserver/server.h 20b5faa 
> 
> 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/20150207/342d397e/attachment.html>