Use ApplicationLauncherJob and CommandLauncherJob.
Also, for services runner pass a KServiceAction to the job so it invokes the appropriate jump list action
Details
Details
- Reviewers
dfaure - Group Reviewers
Plasma - Commits
- R120:6e660cf92d7c: Port services and shell runner away from KRun
Depends on D28295
- Running normal programs still works
- Invoking jump list action entries still works
- Running programs with spaces in their desktop file (checking if that QUrl breaks some encoding) still works
- Running shell commands still works
- Running shell commands in terminal (unchanged) still works
- When program fails to launch, I now get a nice plasma notification
Makes me wonder if we should do the notifyAccessed stuff in the job as well?
Diff Detail
Diff Detail
- Repository
- R120 Plasma Workspace
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
- Adjust to API change in KServiceAction
- Also don't use now deprecated QUrl::queryItemValue