Skip to content

services.redshift: Enable auto-restart#1425

Closed
the-kenny wants to merge 2 commits intoNixOS:masterfrom
the-kenny:fix-redshift
Closed

services.redshift: Enable auto-restart#1425
the-kenny wants to merge 2 commits intoNixOS:masterfrom
the-kenny:fix-redshift

Conversation

@the-kenny
Copy link
Contributor

The redshift service stops working after some time (the program exits
after some hours/days). It looks like these exits are related to calls
to xrandr (for multiple displays) or suspend-to-ram.

The redshift service stops working after some time (the program exits
after some hours/days). It looks like these exits are related to calls
to xrandr (for multiple displays) or suspend-to-ram.

Signed-off-by: Moritz Ulrich <moritz@tarn-vedra.de>
@ocharles
Copy link
Contributor

This will also probably help with redshift starting too quickly - before the display-manager service has fully started and X is ready.

@bjornfor
Copy link
Contributor

If redshift depends on display-manager being fully started, maybe it should have after = [ "display-manager.service" ];?

@ocharles
Copy link
Contributor

Yea, that would be great! I see I only got as far as specifying the requirement on display-manager, but not the ordering. Could you add that in here, @the-kenny ?

@domenkozar
Copy link
Member

+1

Signed-off-by: Moritz Ulrich <moritz@tarn-vedra.de>
@the-kenny
Copy link
Contributor Author

Done. I bet this was also a cause of the issues I (and @DerGuteMoritz) experience.

@bjornfor
Copy link
Contributor

Rebased and pushed to master.

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.

4 participants