-
Notifications
You must be signed in to change notification settings - Fork 115
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
Don't hardcode /usr/include in meson.build #569
Comments
Patches welcome ;-) |
Filed snowballstem/snowball#190 to nudge snowball maintainers. |
Ideally snowball would support installing anything at all, period. However I am slightly curious why you need to specify any include directories at all. Do the compiler builtin defaults not work? That should already have /usr/include, but only on native builds, since in cross builds it will have /path/to/cross-sysroot/usr/include And given that you don't know where snowball is installed without a pkg-config file, it seems 100% pointless to hardcode an arbitrary location that may not be true. |
Right, I'd have thought that the meson header/library checking code would have been sufficient to validate that the library is present. |
If AppStream is build with stemmer enabled, it forcibly looks in
/usr/include
:appstream/meson.build
Line 188 in 1217913
I may have installed stemmer anywhere, and in cross-compiled environments hardcoding this path is absolutely the wrong thing to do as it is the host includes not the target includes.
The text was updated successfully, but these errors were encountered: