Bug 484868 - Power-profiles-daemon service is advertised under a new name starting with v0.20
Summary: Power-profiles-daemon service is advertised under a new name starting with v0.20
Status: RESOLVED FIXED
Alias: None
Product: Powerdevil
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-04-01 12:34 UTC by mrfrh
Modified: 2024-09-13 18:08 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description mrfrh 2024-04-01 12:34:56 UTC
SUMMARY
NOTE - this is not really a bug, but a potential bug as PowerDevil currently relies on the old service name which is only kept for compatibility reasons. The old name might be removed in the future (openSUSE even tried to remove it immediately, see https://bugzilla.suse.com/show_bug.cgi?id=1219956).

Prior to release 0.20, power-profiles-daemon was advertised as 'net.hadess.PowerProfiles'.
Starting with release 0.20 the project has moved under the freedesktop 'Upower' group. The service is
now advertised as 'org.freedesktop.UPower.PowerProfiles' in addition to the previous 'net.hadess.PowerProfiles' for compatibility reasons. See https://gitlab.freedesktop.org/upower/power-profiles-daemon/-/releases

To avoid issues, could the new service name be added to PowerDevil?



SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.0.3
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.3

ADDITIONAL INFORMATION
Comment 1 Jakob Petsovits 2024-04-03 16:25:05 UTC
Thanks, this seems like a good idea and we should check both service names going forward. (Until we can drop support for <0.20 eventually.)
Comment 2 Bug Janitor Service 2024-09-02 16:03:52 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/424
Comment 3 Jakob Petsovits 2024-09-13 18:08:10 UTC
Git commit 223b927a8351f35d7c2c2a70ea6d78568051ba3c by Jakob Petsovits.
Committed on 13/09/2024 at 16:03.
Pushed by jpetso into branch 'master'.

daemon/actions/powerprofile: Use the new D-Bus names for p-p-d

power-profiles-daemon was moved into the UPower namespace,
where it changed its bus, path & interface names from the original
net.hadess.PowerProfiles to org.freedesktop.UPower.PowerProfiles.

These new names are advertised starting with v0.20 released in
February 2024. The old names are still supported at least in the
current v0.21, but we don't know for how long (or on all distros,
given that OpenSuse already tried to remove it early on).

A February 2024 release seems long enough ago that we shouldn't
require support for the old interface anymore. Rather than adding
a second interface and introducing complexity, let's just make
a clean switch to the new name while both are supported.

M  +1    -1    applets/batterymonitor/plugin/powerprofilescontrol.cpp
M  +5    -5    autotests/applets/NetHadessPowerProfiles.py
M  +1    -1    daemon/actions/bundled/CMakeLists.txt
R  +11   -4    daemon/actions/bundled/org.freedesktop.UPower.PowerProfiles.xml [from: daemon/actions/bundled/net.hadess.PowerProfiles.xml - 093% similarity]
M  +4    -4    daemon/actions/bundled/powerprofile.cpp
M  +2    -2    daemon/actions/bundled/powerprofile.h

https://invent.kde.org/plasma/powerdevil/-/commit/223b927a8351f35d7c2c2a70ea6d78568051ba3c