configure: use AC_CHECK_HEADERS to check for endian.h
authorEmil Velikov <emil.velikov@collabora.co.uk>
Fri, 23 Mar 2018 17:37:39 +0000 (17:37 +0000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Fri, 23 Mar 2018 18:12:52 +0000 (18:12 +0000)
commit5a75019ad0270a974788a9b8648ba98ff4203768
tree37df967af74e8e826e1e197ee763724f7eec4c39
parent90f556f0b15a8767ec5b98fa0aea2fbdd352fd5e
configure: use AC_CHECK_HEADERS to check for endian.h

The currently we use the singular CHECK_HEADER combined with explicit
append to the DEFINES variable. That is a legacy misnomer, since it
requires us to add $DEFINES to every piece that we build.

Using the plural version of the helper sets the HAVE_ macro for us, plus
ensures it's passed to the compiler - if config.h is available in there
(not in the case of mesa) otherwise on the command line.

In hindsight, we should replace all the AC_CHECK_{FUNC,HEADER} instances
with the plural version (or even the _ONCE suffixed version) and drop
the DEFINES hacks.

Fixes: cbee1bfb342 ("meson/configure: detect endian.h instead of trying
to guess when it's available")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105717
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Acked-by: Eric Engestrom <eric.engestrom@imgtec.com>
Tested-by: Clayton Craft <clayton.a.craft@intel.com>
configure.ac