D28988: Port hotplug from KRun::runCommand to CommandLauncherJob.

David Faure noreply at phabricator.kde.org
Sun Apr 19 21:43:55 BST 2020


dfaure added a comment.


  As it is currently implemented, CommandLauncherJob is very unlikely to ever fail. It doesn't even fail on non-existing executable in the command (because it delegates to bash) so apart from QProcess cannot fork, it never fails. Probably not worth doing something complicated here then. I'll just use KNotificationJobUiDelegate?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D28988

To: dfaure, davidedmundson, sitter
Cc: broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, 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/20200419/68885a54/attachment.html>


More information about the Plasma-devel mailing list