----------------------------------------------------------- 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 I don't see another way. - Martin Gräßlin On Feb. 9, 2015, 8:10 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122339/ > ----------------------------------------------------------- > > (Updated Feb. 9, 2015, 8:10 p.m.) > > > 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/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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel