-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
meson: unbreak linux/input-event-codes.h detection in some cases #10871
Conversation
What's stopping this from being fixed in |
I've filed https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=267784 |
Wouldn't waf have this same problem or is something different there? |
Looking at this one again. I'd be fine with |
what's the status here? freebsd still uses this patch and would be nice to upstream it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally forgot about this. It looks kind of weird, but I'm fine with it.
@jbeich: just pinging you in case you don't see the comment. |
On DragonFly and FreeBSD native evdev headers are not not be used outside of base system. Linux-compatible headers are provided by evdev-proto package but require extra flags. As evdev-proto lacks pkg-config support use libwayland prefix as approximation. meson.build:974:0: ERROR: C header 'linux/input-event-codes.h' not usable
Pushed as 3bdf702 (got rid of an extra newline). Sorry about all the delay on this! |
Found downstream. When disabling some Meson options
-I/usr/local/include
maybe disappear, breaking configure check.evdev-proto
(a subset of linux-headers) which provides the header on some BSDs doesn't support pkg-config (lacks *.pc file), so use one of wayland libraries as approximate location for now.Note, using
dependencies
asinclude_directories
looks even more hacky:include_directories: include_directories(dependency('wayland-client').get_variable(pkgconfig: 'includedir')))
include_directories: include_directories(get_option('prefix') / 'include')
https://mesonbuild.com/Reference-manual_returned_compiler.html#compilercheck_header