D27216: [KProcessRunner] Improve error messages on failure

[David Faure]

dfaure added a comment.


  Done in D29170 <https://phabricator.kde.org/D29170>. Now we can start discussing the other part of the bug, executable doesn't launch due to missing libs.
  
  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.

REPOSITORY
  R241 KIO

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

To: ahmadsamir, #frameworks, dfaure, davidedmundson, broulik
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200425/46848b4b/attachment.html>