Bug 101803

Summary:	libinput 1.8.0 -- meson build missing --includedir= option present in autotools
Product:	Wayland	Reporter:	Bjørn Lie <bjorn.lie>
Component:	libinput	Assignee:	Wayland bug list <wayland-bugs>
Status:	RESOLVED NOTABUG	QA Contact:
Severity:	enhancement
Priority:	medium	CC:	peter.hutterer
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Bjørn Lie 2017-07-16 13:22:49 UTC

When using autotools to build libinput it is possible to pass --includedir=foo to configure to set a non default folder as include dir.

This possibility seems to be missing from the meson build (or is non advertised).

Please consider making this a possibility when using meson too.

Comment 1 Peter Hutterer 2017-07-17 23:36:07 UTC

mesonconf shows there is a 'includedir' option, so you should be able to pass this into meson with -Dincludedir='path'

If this doesn't work, please file a bug with meson as this is a generic build system option, not one specific to libinput. Thanks

Comment 2 Bjørn Lie 2017-08-30 20:46:31 UTC

So the solution here finally dawned on me. In case someone lands on this bug via search, here goes:

--includedir="%_includedir/%name" \

When passing the "standard" options to meson, they should be passed with --foo and not -Dfoo

--

New build systems are a bugger sometimes ;-)  - but this was indeed not a bug.

Comment 3 Peter Hutterer 2017-08-30 22:10:49 UTC

huh, interesting, thanks for the follow-up. mind you, iirc if you change it with mesonconf later, it should be with -D which is a tad inconsistent. Worth filing a meson bug for, I think.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.