D26066: [KProcessInfo] Add parent PID

[David Faure]

dfaure added a comment.


  Yep, seems fine to me.

REPOSITORY
  R244 KCoreAddons

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

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

D26066: [KProcessInfo] Add parent PID

[Méven Car]

meven added subscribers: dfaure, meven.
meven added a comment.


  In D26066#598036 , @broulik wrote:
  
  > Am now using ksysguard..
  
  
  I think this would makes sense, and is a nice addition to `KProcessInfo`.
  @dfaure an opinion ?

REPOSITORY
  R244 KCoreAddons

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

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

D26066: [KProcessInfo] Add parent PID

[Kai Uwe Broulik]

broulik abandoned this revision.
broulik added a comment.


  Am now using ksysguard..

REPOSITORY
  R244 KCoreAddons

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

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

D26066: [KProcessInfo] Add parent PID

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Frameworks, davidedmundson.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
broulik requested review of this revision.

REVISION SUMMARY
  We basically get this information for free anyway.

TEST PLAN
  - Can now query parent pid in plasma task manager without pulling in ksysguard
  - /proc and ps work, FreeBsd (procstat), and windows untested

REPOSITORY
  R244 KCoreAddons

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

AFFECTED FILES
  autotests/kprocesslisttest.cpp
  src/lib/util/kprocesslist.cpp
  src/lib/util/kprocesslist.h
  src/lib/util/kprocesslist_p.h
  src/lib/util/kprocesslist_unix.cpp
  src/lib/util/kprocesslist_unix_procstat_p.h
  src/lib/util/kprocesslist_win.cpp

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