<table><tr><td style="">ahmadsamir updated this revision to Diff 79399.<br />ahmadsamir retitled this revision from "[KProcessRunner] Improve error reported to user when exit code != 0" to "[KProcessRunner] Improve error messages on failure".<br />ahmadsamir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-zrtv3aqenogw623/">(Show Details)</a><br />ahmadsamir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-22poanlab2gv32e/">(Show Details)</a><br />ahmadsamir 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/D27216">View Revision</a></tr></table><br /><div><div><p>Update and rebase</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">When a process</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Change the message</span> that<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> KRun tried to run fails to launch:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'s displayed when we can't find an executable</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">to mention the word "executable",</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- Tell the user when</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">because</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">executable</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">binary</span> could<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> not</span> be <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">found</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">there</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> - </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but with wrong permissions.<br />
<br />
</span>If we find an executable but it exit<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">ed</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s</span> abnormally (exit code != 0 ),<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">   </span>inform the user about that and suggest he tries running it from terminal<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">   </span>to get more details<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">(Proceed with the sudo commands at your own risk<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> :)</span>):<div style="padding: 8px 0;">...</div>  gwenview from e.g. dolphin, see the error message<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
- break your system slightly:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">   `</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- '</span>sudo chmod a+x /usr/bin/gwenview <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">&</span>& sudo mv /usr/lib64/libKF5Kipi.so.32.0.0 /usr/<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'</span><br />
  <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> </span>then try opening a picture with gwenview from dolphin, see the error message<br />
- Finally<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">: `</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> '</span>sudo mv /usr/libKF5Kipi.so.32.0.0 /usr/lib64/<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">'</span> to un<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">-</span>break your system<br />
<br />
BUG: 415567<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
</span>FIXED-IN: 5.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">67</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">70</span></div></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/D27216?vs=75179&id=79399">https://phabricator.kde.org/D27216?vs=75179&id=79399</a></div></div><br /><div><strong>BRANCH</strong><div><div>l-kprunner-message (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27216">https://phabricator.kde.org/D27216</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/gui/CMakeLists.txt<br />
src/gui/kprocessrunner.cpp</div></div></div><br /><div><strong>To: </strong>ahmadsamir, Frameworks, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>