<table><tr><td style="">dfaure updated this revision to Diff 77682.<br />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><ul class="remarkup-list">

<li class="remarkup-list-item">Don't block in start(), make it fully async</li>

<li class="remarkup-list-item">Add waitForStarted() for KRun (with unittests)</li>

<li class="remarkup-list-item">Add test for non-existing executables, with and without kioexec<ul class="remarkup-list">

<li class="remarkup-list-item">after making sure that the command isn't trivial, by starting with cd KIO::DesktopExecParser helps, though, it returns /bin/sh as such a case So we don't have to do this when the process finishes, we can check it upfront</li>

</ul></li>

<li class="remarkup-list-item">Connect to QProcess::errorOccurred just for debugging (debug output in case the process crashes)</li>

</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D28020?vs=77547&id=77682">https://phabricator.kde.org/D28020?vs=77547&id=77682</a></div></div><br /><div><strong>BRANCH</strong><div><div>2020_ProcessLauncherJob</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>AFFECTED FILES</strong><div><div>KF5KIOConfig.cmake.in<br />

autotests/CMakeLists.txt<br />

autotests/kiotesthelper.h<br />

autotests/processlauncherjobtest.cpp<br />

autotests/processlauncherjobtest.h<br />

src/gui/CMakeLists.txt<br />

src/gui/config-kiogui.h.cmake<br />

src/gui/kprocessrunner.cpp<br />

src/gui/kprocessrunner_p.h<br />

src/gui/processlauncherjob.cpp<br />

src/gui/processlauncherjob.h<br />

src/widgets/CMakeLists.txt<br />

src/widgets/krun.cpp<br />

src/widgets/krun.h<br />

src/widgets/krun_p.h</div></div></div><br /><div><strong>To: </strong>dfaure, apol, davidedmundson, nicolasfella, vkrause, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>