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

check EDID only for load config #174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fe80
Copy link

@fe80 fe80 commented Dec 15, 2019

Load config parse all available and fail when monitor is not present in a configuration:

autorandr -s home
Failed to load profiles (line 416; /home/steffy/.pip/lib/python3.8/site-packages/autorandr.py):
  Failed to find an EDID for output `DP-2-1' in setup file, required as `DP-2-1' is not off in config file. (line 416; /home/steffy/.pip/lib/python3.8/site-packages/autorandr.py)

I move the EDID error only when we want load a profile with an error, and add --force for load and disable unknown screen

@phillipberndt
Copy link
Owner

Thanks!

The intention is that this error message is supposed to show if the config file defines a configuration for an output, but setup does not assign an EDID to it. That can only be the case if you manually edited the file.

Is that the case? Or did this happen without fiddling around with the files? In that case there's another bug, and I'd prefer to fix that one.

@glems2
Copy link

glems2 commented Oct 28, 2022

I had that error recently without having edited any config or setup files.
At least the error message should hint at the problematic profile. Took me some time to find it out by hand.

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