<table><tr><td style="">dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28020">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Would it be possible to expose the finished signal of QProcess in KIO::ApplicationLauncherJob?</p></blockquote>
<p>By design the job finishes before the subprocess finishes (which could be in 3 days, if the user keeps the window open that long).<br />
Also this would only help you for subprocesses started by plasma itself, but not for the case where application A starts application B.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>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.</p></blockquote>
<p>This is exactly what the code in KProcessRunner::terminateStartupNotification is about.<br />
If you're not on X11, then this code needs to be extended to notify plasma another way about terminating startup notification.<br />
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.</p>
<p>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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28020">https://phabricator.kde.org/D28020</a></div></div><br /><div><strong>To: </strong>dfaure, apol, davidedmundson, nicolasfella, vkrause, broulik<br /><strong>Cc: </strong>jbbgameich, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>