[KProcessRunner] Use only executable name for scope
ClosedPublic

Authored by broulik on Apr 27 2020, 1:03 PM.

Details

Summary

Slashes aren't allowed in scope names, so when a full executable path is launched, e.g. ´/usr/bin/foo` it would fail to be scoped.
It's kinda wrong, too, but better than not scoping it at all.

Test Plan

In D29226 I now have my session restored apps in scopes, though they're naturally not the same ones as when I have a desktop entry associated with the launch job

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Apr 27 2020, 1:03 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 27 2020, 1:03 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Apr 27 2020, 1:03 PM
davidedmundson accepted this revision.May 19 2020, 9:54 AM
This revision is now accepted and ready to land.May 19 2020, 9:54 AM
This revision was automatically updated to reflect the committed changes.