flathub / io.github.pwr_solaar.solaar

https://flathub.org/apps/details/io.github.pwr_solaar.solaar
2 stars 1 forks source link

Add --restart-on-wake-up support #7

Closed akvadrako closed 6 months ago

akvadrako commented 7 months ago

In my experience, solaar needs to re-configure devices after suspend, so I use this experimental flag:

io.github.pwr_solaar.solaar -d --restart-on-wake-up
...
2024-01-30 09:37:02,383,383  WARNING [MainThread] solaar.upower: failed to register suspend/resume callbacks

This requires a new talk permission on the system DBus:

org.freedesktop.UPower

It would be nice if that flag was on by default but at least it should be safe to add that DBus permission to the manifest.

proletarius101 commented 6 months ago

Can you explain more about it? Should we use this flag by default?

akvadrako commented 6 months ago

As I understand, after waking up from sleep the devices need their DPI re-adjusted and this helps with that. I'm not totally sure though and not sure what it isn't on by default.

proletarius101 commented 6 months ago

As it seems to be experimental, I won't turn it on by default in Flatpak. But the 2 buses will be open. However, it's blocked by https://github.com/pwr-Solaar/Solaar/pull/2296. You'll need to override and add the 2 permissions on your machine for now.