[Intel-gfx] [PATCH i-g-t 02/12] build: Nuke #ifdef HAVE_CONFIG_H cargo-cult
Daniel Vetter
daniel at ffwll.ch
Mon Sep 4 08:41:38 UTC 2017
On Sun, Sep 03, 2017 at 10:29:06AM +0100, Chris Wilson wrote:
> Quoting Daniel Vetter (2017-09-02 18:03:56)
> > We have it. Daniel Stone said it comes from the X11 transition to the
> > modular build.
>
> What? It's from AC_CONFIG_HEADERS([config.h])
>
> We tell autoconf to build it so it doesn't have to put all the build
> flags on the command line.
Yes I know where it's from. What I mean is why do we include-guard this,
and Daniel Stone said very old versions of X didn't have config.h (because
they used imake or whatever the horror was again).
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list