Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add upower_power_profiles_daemon template #201

Merged
merged 2 commits into from
Feb 23, 2024
Merged

Add upower_power_profiles_daemon template #201

merged 2 commits into from
Feb 23, 2024

Conversation

martinpitt
Copy link
Owner

This works like the existing power_profiles_daemon template, but for the
new D-Bus name/object path introduced in
https://gitlab.freedesktop.org/upower/power-profiles-daemon/-/releases/0.20

Enable integration test on Ubuntu, which has version 0.13 in 22.04 LTS.

…ofiles"

This was too quick: Except for `powerprofilesctl`, all "actual"
consumers are currently still using the old name.

This reverts commit 8cec9a1.
This works like the existing power_profiles_daemon template, but for the
new D-Bus name/object path introduced in
https://gitlab.freedesktop.org/upower/power-profiles-daemon/-/releases/0.20

Enable integration test on Ubuntu, which has version 0.13 in 22.04 LTS.
@martinpitt
Copy link
Owner Author

@3v1n0 @hadess FYI

@martinpitt martinpitt merged commit b24f7b7 into main Feb 23, 2024
28 checks passed
@martinpitt martinpitt deleted the power-profiles branch February 23, 2024 13:04
@superm1
Copy link
Contributor

superm1 commented Feb 23, 2024

I might have broken your plan for it with the proposed change to fix your bug report https://gitlab.freedesktop.org/upower/power-profiles-daemon/-/issues/139. Can you look at that please?

@martinpitt
Copy link
Owner Author

@superm1 I saw -- that'll require some adjustment to the parsing indeed. I'll work on a fix ASAP, thanks for the warning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants