Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Compatibility with Wayland/new systems #36

Open
bendahl opened this issue May 10, 2023 · 2 comments
Open

Compatibility with Wayland/new systems #36

bendahl opened this issue May 10, 2023 · 2 comments

Comments

@bendahl
Copy link
Owner

bendahl commented May 10, 2023

It appears that on some more recent systems the uinput initialization does not work as expected (see #34). This was pointed out by @gslandtreter. The issue appeared on a system using Wayland. However, it should also be investigated whether the kernel version is a factor here. Once the root cause is confirmed this should be fixed in a backward-compatible manner, if possible without breaking the existing API.

@gslandtreter
Copy link
Contributor

Hi @bendahl, thanks for pointing it out. Indeed I submitted the PR and it worked fine on wayland, but I didn't check if/when the ioctl API changed or whether the change was not backwards compatible with older kernels.

I can help you with this one.
The ideal solution would be to check the ioctl API / kernel version and make the calls accordingly. Agreed?

@bendahl
Copy link
Owner Author

bendahl commented May 12, 2023

Sounds good! I think that performing an internal version check is definitely the way to go. Thanks for offering your help on this one, by the way. This is much appreciated!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants