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

do not fail on EACCES when trying to open() evdevs #436

Closed
wants to merge 2 commits into from

Conversation

tempura-san
Copy link
Contributor

(hopefully) improved fix of problem described in #431

@tempura-san
Copy link
Contributor Author

The second commit could be omitted. I tend to finalise every statement, but the definition of PRINT_ERROR_ERRNO() allows the semicolon to be optional as it is already present in the macro.

@matlo
Copy link
Owner

matlo commented Feb 9, 2017

Are you sure the errno value was EACCES = "Permission denied"?

A similar issue was reported on the forum: https://forum.gimx.fr/viewtopic.php?f=5&t=1996
In this case errno was EPERM = "Operation not permitted".

I think I will make all errors to be ignored (but still reported in the messages). Someone may want to apply different ownership and permissions on devices.

@matlo matlo closed this Feb 9, 2017
@tempura-san
Copy link
Contributor Author

I will check the error code once I have the computer back at home - likely tomorrow.

@matlo
Copy link
Owner

matlo commented Feb 9, 2017

This should be fixed by #437.

@tempura-san
Copy link
Contributor Author

@matlo I messed it up. In fact it was EPERM (=Operation not permitted). 😓 I have the German locale active, so it was a bit misleading. Sorry for the confusion.
The latest change #437 fixes the issue for me. The two offending devices (/dev/input/event9 and /dev/input/js0) are listed in the console failing with EPERM, but GIMX starts nonetheless.
So thank you for your help and support!

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

2 participants