Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-01-31 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/ --- Review request for Plasma and Martin Gräßlin. Bugs: 189456 https://bu

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/#review75190 --- is that for the case that no logind is used? Or should ksmserv

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-07 Thread 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 Pl

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-08 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/#review75659 --- libkworkspace/kworkspace.cpp

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-09 Thread Kai Uwe Broulik
--- 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

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-09 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/#review75755 --- Ship it! I'm not 100 % happy with the blocking dbus call, but

Re: Review Request 122339: Add KWorkSpace::isShuttingDown()

2015-02-10 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122339/ --- (Updated Feb. 10, 2015, 9:15 a.m.) Status -- This change has been ma