Differential D22327

[libnotificationmanager] Port to lighter KProcessList
ClosedPublic
Actions

Authored by davidedmundson on Jul 8 2019, 2:14 PM.

Details

Reviewers
broulik
Group Reviewers
Plasma
Commits
R120:d14aca8cd861: [libnotificationmanager] Port to lighter KProcessList
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.

Diff Detail

Repository
R120 Plasma Workspace
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13707
Build 13725: arc lint + arc unit
davidedmundson created this revision.Jul 8 2019, 2:14 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 8 2019, 2:14 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Jul 8 2019, 2:14 PM
davidedmundson updated this revision to Diff 61345.Jul 8 2019, 2:17 PM

remove from cmake

broulik requested changes to this revision.EditedJul 8 2019, 2:19 PM
broulik added a subscriber: broulik.

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

notify-send foo bar -a "" should say just "notify-send"

This revision now requires changes to proceed.Jul 8 2019, 2:19 PM
broulik accepted this revision.Jul 31 2019, 12:32 PM

With D22471 does what it should

This revision is now accepted and ready to land.Jul 31 2019, 12:32 PM
Closed by commit R120:d14aca8cd861: [libnotificationmanager] Port to lighter KProcessList (authored by davidedmundson). · Explain WhyAug 15 2019, 3:07 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 61345

View Options

libnotificationmanager/CMakeLists.txt

Loading...
View Options

libnotificationmanager/utils.cpp

Loading...