D23944: Move more functionality from KRun::runCommand/runApplication to KProcessRunner

[David Faure]

dfaure created this revision.
dfaure added reviewers: vkrause, nicolasfella, broulik.
Herald added a project: Frameworks.
Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks.
dfaure requested review of this revision.

REVISION SUMMARY
  - Make that functionality QWidget-independent
  - Emit errors as a signal rather than a messagebox. KRun still connects
  
  to the signal and shows messageboxes, so there's no behaviour change for
  KRun users.
  
  This is one more step in the direction of OpenUrlJob+KProcessRunner,
  both in KIOGui.
  
  Next steps: turning it into a KJob, then renaming it to
  e.g. KIO::ProcessLauncherJob

TEST PLAN
  I'll write a unittest after conversion to KJob.
  For now I used the GUI `kruntest` program.

REPOSITORY
  R241 KIO

BRANCH
  1_kprocesrunner

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

AFFECTED FILES
  src/widgets/krun.cpp
  src/widgets/krun_p.h

To: dfaure, vkrause, nicolasfella, broulik
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190914/7df447e8/attachment.html>