Move kactivitymanagerd to libexec and remove manual daemon management
ClosedPublic

Authored by davidedmundson on Oct 1 2018, 8:49 AM.

Details

Summary

Since it was made DBus activatable having any other code to start/stop
the daemon doesn't really make any sense. For that reason I've
moved it to libexec.

Also the original code is somewhat questionable. It creates a whole qapp
instance, checks if we're registered then forks itself to spawn a new
version of the app that then registers.

KDBusService already does all this in a simpler faster non-racey way.

Test Plan

Installed
Still got kactivitymanagerd

Diff Detail

Repository
R161 KActivity Manager Service
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3388
Build 3406: arc lint + arc unit
davidedmundson created this revision.Oct 1 2018, 8:49 AM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 1 2018, 8:49 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Oct 1 2018, 8:49 AM
mart accepted this revision.Oct 1 2018, 11:52 AM
This revision is now accepted and ready to land.Oct 1 2018, 11:52 AM
This revision was automatically updated to reflect the committed changes.