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

Reply via email to