Review Request 126268: Move shutdown scripts into ksmserver cleanup

David Edmundson david at davidedmundson.co.uk
Sun Dec 13 16:29:54 UTC 2015


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

(Updated Dec. 13, 2015, 4:29 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 1dc7f11692a4096b9815ae24f7be9cec10f8f7a5 by David Edmundson to branch Plasma/5.5.


Repository: plasma-workspace


Description
-------

Shutdown scripts are done by startkde after ksmserver quits. Which never
happens because we've told systemd to shutdown.

Old systems worked because they used to communicate with the display
manager which then closed us before shutting down, this is no longer the
case.

This moves handling the shutdown scripts into ksmserver (which already
handles startup scripts) and runs them before asking logind to shutdown.

BUG: 356190


Diffs
-----

  ksmserver/server.h 1a2f0810c53d508d7d721fb4be1c25f0585c0602 
  ksmserver/server.cpp 9477e542399ad65f993e581fc4212883f282b70e 
  startkde/startkde.cmake 41a8975cce1fb2a4e7a034e697ce6e2cc59d5b1e 
  startkde/startplasma.cmake 8360a636d3f68c957a15158484360a611cfe3ff8 

Diff: https://git.reviewboard.kde.org/r/126268/diff/


Testing
-------

had a shutdown script to touch a file in my home, worked for shutdown and logout.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151213/7fd824ae/attachment.html>


More information about the Plasma-devel mailing list