Diffusion KIO 74cf3da3640c

New class ProcessLauncherJob in KIOGui

Authored by dfaure on Mar 15 2020, 9:54 PM.

Description

New class ProcessLauncherJob in KIOGui

Summary:
This is meant to replace KRun::runApplication/runService,
without the QWidget dependency, for most use cases. One exception:
applications who want to allow the user to make desktop files executable
(this requires messageboxes, and therefore is still in KRun).

The code is mostly based on KRun's internal KProcessRunner class,
now moved to KIOGui, but still private.

Next step: also routing KRun::runCommand via ProcessLauncherJob,
and then writing OpenUrlJob on top of ProcessLauncherJob (but
runUrl calling KOpenWithDialog is a problem in KIOGui...).

Test Plan: see new unittest (which is based on krununittest)

Reviewers: apol, davidedmundson, nicolasfella, vkrause, broulik

Reviewed By: davidedmundson

Subscribers: kde-frameworks-devel

Tags: Frameworks

Differential Revision: https://phabricator.kde.org/D28020

Details

Committed
dfaureMar 18 2020, 8:16 PM
Reviewer
davidedmundson
Differential Revision
D28020: New class ProcessLauncherJob in KIOGui
Parents
R241:dc8aaa7db08a: SVN_SILENT made messages (.desktop file) - always resolve ours
Branches
Unknown
Tags
Unknown