[Mesa-dev] [PATCH RESEND 2/3] gallium/state_trackers/egl: use X11_INCLUDES rather than X11_CFLAGS

Gaetan Nadon memsize at videotron.ca
Fri Sep 27 14:42:27 PDT 2013


The X11_CFLAGS variable is undefined (not defined in config.status).
It appears the intent was to use X11_INCLUDES defined in configure.ac.
It is used for building the code in the x11 subdir.

The build does not fail on this one as LIBDRM_CFLAGS happens to have
the inludedir value as the one for X11. It will not always be the case.
The option --enable-gallium-egl is required durimg configuration.

Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
---
 src/gallium/state_trackers/egl/Makefile.am |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/egl/Makefile.am b/src/gallium/state_trackers/egl/Makefile.am
index 126fafc..ecae3e6 100644
--- a/src/gallium/state_trackers/egl/Makefile.am
+++ b/src/gallium/state_trackers/egl/Makefile.am
@@ -49,7 +49,7 @@ libegl_la_SOURCES += \
 	x11/x11_screen.c \
 	x11/dri2.c
 AM_CFLAGS += \
-	$(X11_CFLAGS) \
+	$(X11_INCLUDES) \
 	$(LIBDRM_CFLAGS) \
 	$(DRI2PROTO_CFLAGS)
 AM_CPPFLAGS += \
-- 
1.7.9.5



More information about the mesa-dev mailing list