D22327: [libnotificationmanager] Port to lighter KProcessList

[David Edmundson]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:d14aca8cd861: [libnotificationmanager] Port to lighter 
KProcessList (authored by davidedmundson).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D22327?vs=61345=63810#toc

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D22327?vs=61345=63810

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

AFFECTED FILES
  libnotificationmanager/CMakeLists.txt
  libnotificationmanager/utils.cpp

To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D22327: [libnotificationmanager] Port to lighter KProcessList

[Kai Uwe Broulik]

broulik accepted this revision.
broulik added a comment.
This revision is now accepted and ready to land.


  With D22471  does what it should

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

D22327: [libnotificationmanager] Port to lighter KProcessList

[Kai Uwe Broulik]

broulik requested changes to this revision.
broulik added a comment.
This revision now requires changes to proceed.


  This now returns the entire command line, not just the process name

REPOSITORY
  R120 Plasma Workspace

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

To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

D22327: [libnotificationmanager] Port to lighter KProcessList

[David Edmundson]

davidedmundson updated this revision to Diff 61345.
davidedmundson added a comment.


  remove from cmake

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D22327?vs=61344=61345

BRANCH
  master

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

AFFECTED FILES
  libnotificationmanager/CMakeLists.txt
  libnotificationmanager/utils.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart

D22327: [libnotificationmanager] Port to lighter KProcessList

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  KSysGuard fetches everything you could possibly need from a process.
  
  If we just need a name of a process, we can use the new KProcessList API
  in KCoreAddons.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  libnotificationmanager/utils.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart