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

XRandR: Allow multiple but not all CRTCs to be redshifted #421

Merged
merged 1 commit into from Jan 14, 2017

Conversation

jonls
Copy link
Owner

@jonls jonls commented Jan 8, 2017

Based on #81.

Previously only one CRTC could be set in the configuration file for
redshifting when XRandR mechanism was being used. That is fine for
a setup with two displays but breaks when three or more displays
are in use and one of those shouldn't be redshifted (e.g. two
computer displays and one TV connected to the computer).

The config value 'crtc' for method xrandr can now be entered as
comma separated list of multiple CRTCs. All CRTCs in the list will
be redshifted while all those not in the list will not be touched.
@lennartS
Copy link
Contributor

Tested and confirmed as working on my machine with 6 CRTCs. Selection of multiple CRTCs works, an error message is printed if the given CRTC does not exist.

@jonls jonls merged commit 460cd9c into master Jan 14, 2017
@jonls jonls deleted the randr-adjust-multiple branch January 14, 2017 17:00
@davorb
Copy link

davorb commented Feb 17, 2017

Sorry if I'm being stupid, but how do I actually use this? I tried building the source and adding a crtc and screen setting to my redshift.conf-file, but that just told me "Unknown configuration setting".

@lennartS
Copy link
Contributor

@davorb, could it be that you did not put the settings inside a [randr] section? I have the following in my config file:

[randr]
screen=0
crtc=0,2

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.

None yet

3 participants