D29228: [KProcessRunner] Use only executable name for scope

[Kai Uwe Broulik]

This revision was automatically updated to reflect the committed changes.
Closed by commit R241:19d23d386cf7: [KProcessRunner] Use only executable name 
for scope (authored by broulik).

REPOSITORY
  R241 KIO

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29228?vs=81338&id=83058

REVISION DETAIL
  https://phabricator.kde.org/D29228

AFFECTED FILES
  src/gui/kprocessrunner.cpp

To: broulik, #frameworks, #plasma, davidedmundson
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29228: [KProcessRunner] Use only executable name for scope

[David Edmundson]

davidedmundson accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D29228

To: broulik, #frameworks, #plasma, davidedmundson
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29228: [KProcessRunner] Use only executable name for scope

[Kai Uwe Broulik]

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  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