Skip to content

Suppress sleep on linux using dbus inhibitor lock#1137

Merged
pifopi merged 10 commits intoPokemonAutomation:mainfrom
ConnorC432:sleep
Mar 23, 2026
Merged

Suppress sleep on linux using dbus inhibitor lock#1137
pifopi merged 10 commits intoPokemonAutomation:mainfrom
ConnorC432:sleep

Conversation

@ConnorC432
Copy link
Contributor

#535
Sleep controller implementation (3f812f2) for linux that allows for both the screen to be kept on all the time, and for the screen to be turned off while preventing sleep.

@Mysticial
Copy link
Collaborator

Thank you for doing this!

This looks fine. I just need to figure out the Linux CI for this. (and get someone to test it) It looks like we need to install the sdbus package.

Lemme ask @pifopi and @Koi-3088 since they know the CI better than me.

@pifopi
Copy link
Member

pifopi commented Mar 20, 2026

I'm attempting this fix ConnorC432#1 Should be trivial but let's wait for CI to run before merging !

@ConnorC432
Copy link
Contributor Author

i have bundled libsdbus-c++ 2.2.1, and managed to build it on ubuntu 24. It just requires the libsystemd-dev package

@Mysticial
Copy link
Collaborator

Verbally approving for the changes to our code. I'll let pif do the final merge when CI when everything is clear.

@pifopi
Copy link
Member

pifopi commented Mar 21, 2026

All good for me, I already approved! There is a merge conflict though

@pifopi pifopi merged commit d2aa35d into PokemonAutomation:main Mar 23, 2026
7 checks passed
@ConnorC432 ConnorC432 deleted the sleep branch March 24, 2026 08:50
Koi-3088 pushed a commit that referenced this pull request Mar 26, 2026
* suppress sleep on linux using dbus inhibitor lock

* Install libsdbus-c++-dev

* use bundled libsdbus-c++ library

* sdbus-c++ overload wrapper

* workflow libsystemd-dev

* always use bundled sdbus-c++

* Revert "sdbus-c++ overload wrapper"

This reverts commit 92dacba.

* fix CMakeLists.txt

---------

Co-authored-by: pifopi <[email protected]>
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.

3 participants