<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/D27216">View Revision</a></tr></table><br /><div><div><p>Done in <a href="https://phabricator.kde.org/D29170" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D29170</a>. Now we can start discussing the other part of the bug, executable doesn't launch due to missing libs.</p>

<p>Ah BTW the unittest leads to "cp" failing because it deletes the temporary dir under cp's feet. This could be fixed, but in any case, we don't want to pop up an error message on every process that terminates with a non-0 exit code. Imagine launching firefox, using it for 2 hours, then it crashes, and you get - in addition to drkonqi - that messagebox saying "It terminated abnormally. For more details try running it from a terminal emulator (e.g. Konsole)" ....  I think at most we want to let the job run for 5 more seconds to handle "crash on startup", but not react on longer-running programs that terminate abnormally.</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/D27216">https://phabricator.kde.org/D27216</a></div></div><br /><div><strong>To: </strong>ahmadsamir, Frameworks, dfaure, davidedmundson, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>