D27629: [plasma-session] Load startup and shutdown on demand

David Edmundson noreply at phabricator.kde.org
Mon Feb 24 17:56:50 GMT 2020


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200224/e7395041/attachment.html>


More information about the Plasma-devel mailing list