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

Reply via email to