broulik created this revision. broulik added reviewers: Frameworks, Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. broulik requested review of this revision.
REVISION 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 <https://phabricator.kde.org/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 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29228 AFFECTED FILES src/gui/kprocessrunner.cpp To: broulik, #frameworks, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns