<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/D28201">View Revision</a></tr></table><br /><div><div><p>(technically, connecting to &KJob::result is missing in the dolphin port, but unlike ProcessLauncherJob, CommandLauncherJob is very unlikely to fail. QProcess will only fail to start if we're out of resources for forking or if <tt style="background: #ebebeb; font-size: 13px;">sh</tt> can't be executed... we don't detect bad shell commands, that happens after QProcess::start(). I wonder if it's worth connecting to the result signal of a CommandLauncherJob....)</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/D28201">https://phabricator.kde.org/D28201</a></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>