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.

Sign up for GitHub

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

Jump to bottom

README: fix #include path to <hidapi/hidapi.h> #323

Closed
wants to merge 1 commit into from
Closed

README: fix #include path to <hidapi/hidapi.h> #323

wants to merge 1 commit into from

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Aug 17, 2021

The build system has always installed the hidapi.h header to
<hidapi/hidapi.h> relative to the installation prefix.

@Be-ing
README: fix #include path to <hidapi/hidapi.h>
1b20d20 
The build system has always installed the hidapi.h header to
<hidapi/hidapi.h> relative to the installation prefix.
@Be-ing Be-ing mentioned this pull request Aug 17, 2021
Closed
@Be-ing
Copy link
Contributor Author

Be-ing commented Aug 17, 2021

Wait, the pkgconfig file is doing some weird stuff:

 ❯ cat /usr/lib64/pkgconfig/hidapi-libusb.pc
prefix=/usr
exec_prefix=/usr
libdir=/usr/lib64
includedir=/usr/include

Name: hidapi-libusb
Description: C Library for USB HID device access from Linux, Mac OS X, FreeBSD, and Windows. This is the libusb implementation.
Version: 0.10.1
Libs: -L${libdir} -lhidapi-libusb
Cflags: -I${includedir}/hidapi

includedir=/usr/include

That's fine.

Cflags: -I${includedir}/hidapi

This is unexpected.

@Be-ing Be-ing closed this Aug 17, 2021
@Youw
Copy link
Member

Youw commented Aug 17, 2021

Cflags: -I${includedir}/hidapi

This is unexpected.

That is expected.

@mcuee mcuee added the documentation Improvements or additions to documentation label Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Be-ing @Youw @mcuee