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

Update hid.c #260

Merged
merged 2 commits into from
May 5, 2021
Merged

Update hid.c #260

merged 2 commits into from
May 5, 2021

Conversation

MonkeybreadSoftware
Copy link
Contributor

Put back in HIDIOCSFEATURE define as it's missing for us.

Put back in HIDIOCSFEATURE define as it's missing for us.
@MonkeybreadSoftware
Copy link
Contributor Author

We build hidapi here for 11 platforms. Lot's of combinations of compilers and platforms, but not all have the HIDIOCSFEATURE define yet.

@z3ntu
Copy link
Collaborator

z3ntu commented Apr 8, 2021

You're building for kernels older than 2.6.39? We don't have support for those anymore in hidapi, so even if it builds it probably doesn't work at runtime.

@MonkeybreadSoftware
Copy link
Contributor Author

The build machine is old to avoid some libc dependencies.
The actual machine running the code is usually newer.

And yes, I know we should update it to something newer.

linux/hid.c Outdated Show resolved Hide resolved
@Youw Youw merged commit 6f0056e into libusb:master May 5, 2021
Copy link
Contributor Author

@MonkeybreadSoftware MonkeybreadSoftware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@mcuee mcuee added the hidraw Related to Linux/hidraw backend label Jun 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hidraw Related to Linux/hidraw backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants