This will allow kwin to grant permission to powerdevil which no other
application should have. Previously used kded module is now replaced by
the application which installs in LIBEXEC dir and started by desktop
file in /etc/xdg/autostart
Details
Details
- Reviewers
broulik graesslin - Group Reviewers
Plasma - Commits
- R122:a5a325e20c15: Make powerdevil normal executable instead of kded module
Restarted session, powerdevil starts and all functions work perfectly
Diff Detail
Diff Detail
- Repository
- R122 Powerdevil
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Something is wrong with this on wayland, I don't see battery and brightness applet and powerdevil executable seems to exit safely if I remove it.. Here is log of powerdevil executable https://ptpb.pw/9Jtd..
Comment Actions
- Don't install in kf5 libexec dir
- Autostart powerdevil only in KDE
- Make PowerDevilApp QGuiApplication instead of QObject
Comment Actions
I just realized that this breaks the shortcuts, and I need to migrate older shortcuts over.
Comment Actions
read shortcuts from kded5 component, unfortunately its close to impossible to migrate shortcuts using kconf_update
daemon/powerdevilapp.cpp | ||
---|---|---|
137–151 | Should it have KCrash::setFlags(AutoRestart) ? |