D28020: New class ProcessLauncherJob in KIOGui

[David Edmundson]

davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kprocessrunner.cpp:194
> +{
> +    return m_process->waitForStarted();
> +}

It's the DBus calls that come before start that I want to get async, not the tiny bit between fork and the child process exec()'ing.

Obviously we can do that piecemeal later, and it isn't a reason to delay this.

I'm not sure how much of that we'll be able to get both async and into this "waitForStarted" pattern without event loops, but worst case we can do it for KF6. It's all a bit frustrating as FWICT no-one even uses the returned PID.

> krun.cpp:488
>  // This code is also used in klauncher.
> -// TODO: move this to KProcessRunner
> +// TODO: port klauncher to KIOGuiPrivate::checkStartupNotify once this lands
> +// TODO: then deprecate this method, and remove in KF6

or we can just ignore it if we're phasing out klauncher anyway?

REPOSITORY
  R241 KIO

BRANCH
  2020_ProcessLauncherJob

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

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