davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. davidedmundson requested review of this revision.
REVISION SUMMARY Currently startplasma spawns plasma-session then sits around waiting for that to finish plasma-session spawns all the startup then also just sits around doing nothing This patch makes plasma-session spawn all the startup and then quit. It also splits the owner of the org.kde.shutdown interface to be on demand. plasma-shutdown asks ksmserver to quit and then if applicable runs the shutdown scripts or not. Startplasma then knows when to exit by monitoring the DBus service status directly. The benefits are that we save some resources by not needing plasma-session lingering about. It also means the shutdown interface is re-usable as-is when the pending systemd startup method is used. TEST PLAN Logged in and: - ran killall ksmserver, session ended as before - logged out and cancelled due to unsaved changes - logged out and completed logout - logged out and rebooted REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D27629 AFFECTED FILES ksmserver/CMakeLists.txt libkworkspace/CMakeLists.txt startkde/CMakeLists.txt startkde/plasma-session/CMakeLists.txt startkde/plasma-session/main.cpp startkde/plasma-session/org.kde.Shutdown.xml startkde/plasma-session/shutdown.cpp startkde/plasma-session/shutdown.h startkde/plasma-session/startup.cpp startkde/plasma-shutdown/CMakeLists.txt startkde/plasma-shutdown/main.cpp startkde/plasma-shutdown/org.kde.Shutdown.xml startkde/plasma-shutdown/shutdown.cpp startkde/plasma-shutdown/shutdown.h startkde/startplasma-wayland.cpp startkde/startplasma-x11.cpp startkde/startplasma.cpp To: davidedmundson, #plasma Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart