Review Request 124838: Fix ksmserver launching shell scripts
David Edmundson
david at davidedmundson.co.uk
Thu Aug 20 14:15:53 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124838/
-----------------------------------------------------------
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
-----
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/36c0fe6f/attachment.html>
More information about the Plasma-devel
mailing list