Review Request 124838: Fix ksmserver launching shell scripts

[David Edmundson]

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

(Updated Aug. 20, 2015, 2:29 p.m.)


Review request for Plasma.


Repository: plasma-workspace


Description
-------

QProcess::startDetached doesn't work on shell scripts, presumably
because it tries to daemonize them.

Porting to QProcess seems to fix this. As KSMServer stays around forever
this shouldn't result in any behavioural change.


Diffs (updated)
-----

  ksmserver/startup.cpp bd80417b25325214199a1cc109f6ea2b2a24ce98 

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


Testing
-------

added some shell scripts with
#/bin/sh
echo hello > /tmp/hello

was greeted pleasantly.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150820/35b4b8d5/attachment.html>