D28020: New class ProcessLauncherJob in KIOGui

[David Faure]

dfaure added a comment.


  > Would it be possible to expose the finished signal of QProcess in KIO::ApplicationLauncherJob?
  
  By design the job finishes before the subprocess finishes (which could be in 3 days, if the user keeps the window open that long).
  Also this would only help you for subprocesses started by plasma itself, but not for the case where application A starts application B.
  
  > I need something like this to close the startup feedback in the Plasma Mobile shell (a fullscreen overlay which shows that an app is starting) when the app crashed.
  
  This is exactly what the code in KProcessRunner::terminateStartupNotification is about.
  If you're not on X11, then this code needs to be extended to notify plasma another way about terminating startup notification.
  I'm no expert on non-X11 startup notification. If there's a cross-desktop spec then it could be used here. Otherwise, I'm OK with a plasma-specific DBus call in there, it won't harm other environments.
  
  By doing it there, it will work from any application. It's "apps talking to the workspace", while giving you access to QProcess::finished would only work for the specific case of the workspace itself starting other apps.

REPOSITORY
  R241 KIO

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

To: dfaure, apol, davidedmundson, nicolasfella, vkrause, broulik
Cc: jbbgameich, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200418/cae98cff/attachment-0001.html>