Summary: | Autostart, stderr is closed if daemonized (conky) | ||
---|---|---|---|
Product: | [Plasma] ksmserver | Reporter: | Maximiliano Curia <maxy> |
Component: | general | Assignee: | Lubos Lunak <l.lunak> |
Status: | RESOLVED FIXED | ||
Severity: | minor | ||
Priority: | NOR | ||
Version: | 5.8.0 | ||
Target Milestone: | --- | ||
Platform: | Debian unstable | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/plasma-workspace/da44dd6eae8b9ef4161680efcb9319a7267fe77e | Version Fixed In: |
Description
Maximiliano Curia
2016-10-12 08:12:39 UTC
https://commits.kde.org/plasma-workspace/eadaf1cd6f55e6092deb2114c5477559ebf063e0 https://commits.kde.org/plasma-workspace/0377d6a992725e0b0b469b87d17343fa3730b829 Git commit 0f19e92f3e85d064de9cebf280fa8e085485c2e0 by Luboš Luňák. Committed on 21/04/2017 at 11:59. Pushed by lunakl into branch 'Plasma/5.8'. launch autostart apps in ksmserver using KRun, not QProcess The "move" (copy in fact) of the autostart code from klauncher simply switched to QProcess. This resulted in a number of problems, as the incorrect handling of .desktop file command if it had placeholders (worked around already), kdeinit is no longer used (not sure how much that matters nowadays) and finally QProcess simply eats all stdout/stderr of all such processes (which, when daemonized, can lead even to the processes getting SIGPIPE'd). If the application to start is a KService, simply launch it as such. Related: bug 369391 M +3 -13 ksmserver/startup.cpp https://commits.kde.org/plasma-workspace/0f19e92f3e85d064de9cebf280fa8e085485c2e0 Git commit da44dd6eae8b9ef4161680efcb9319a7267fe77e by Luboš Luňák. Committed on 21/04/2017 at 12:00. Pushed by lunakl into branch 'Plasma/5.9'. launch autostart apps in ksmserver using KRun, not QProcess The "move" (copy in fact) of the autostart code from klauncher simply switched to QProcess. This resulted in a number of problems, as the incorrect handling of .desktop file command if it had placeholders (worked around already), kdeinit is no longer used (not sure how much that matters nowadays) and finally QProcess simply eats all stdout/stderr of all such processes (which, when daemonized, can lead even to the processes getting SIGPIPE'd). If the application to start is a KService, simply launch it as such. Related: bug 369391 M +3 -13 ksmserver/startup.cpp https://commits.kde.org/plasma-workspace/da44dd6eae8b9ef4161680efcb9319a7267fe77e Git commit c7d21ae2b834bd101d8dba1edaea05edcdb6e8f5 by Luboš Luňák. Committed on 21/04/2017 at 12:00. Pushed by lunakl into branch 'Plasma/5.9'. use KProcess instead of QProcess to launch apps in ksmserver Follow-up to the previous commit. KProcess has saner defaults for simply launching an app and being done with it, no lost stdout/stderr or even SIGPIPE. Related: bug 369391 M +3 -2 ksmserver/startup.cpp https://commits.kde.org/plasma-workspace/c7d21ae2b834bd101d8dba1edaea05edcdb6e8f5 |