On Jan 31, 2013 2:51 AM, "Michel Dänzer" <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote: > > From: Michel Dänzer <<a href="mailto:michel.daenzer@amd.com">michel.daenzer@amd.com</a>> > > GLX uses mapi/glapi/<a href="http://libglapi.la">libglapi.la</a>, which is only built for OpenGL. > > Signed-off-by: Michel Dänzer <<a href="mailto:michel.daenzer@amd.com">michel.daenzer@amd.com</a>> > --- > <a href="http://configure.ac">configure.ac</a> | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/<a href="http://configure.ac">configure.ac</a> b/<a href="http://configure.ac">configure.ac</a> > index cfd52bf..62cc32b 100644 > --- a/<a href="http://configure.ac">configure.ac</a> > +++ b/<a href="http://configure.ac">configure.ac</a> > @@ -701,6 +701,14 @@ if test "x$enable_dri$enable_xlib_glx" = xyesyes; then > AC_MSG_ERROR([DRI and Xlib-GLX cannot be built together]) > fi > > +# Disable GLX if OpenGL is not enabled > +if test "x$enable_glx" = xyes -a \ > + "x$enable_opengl" = xno; then > + AC_MSG_WARN([OpenGL not enabled, disabling GLX]) > + enable_glx=no > + enable_xlib_glx=no > +fi Seems correct. I might prefer this to error, but warn is probably good enough. The option to disable opengl didn't exist back when these checks were written. Reviewed-by: Dan Nicholson <<a href="mailto:dbn.lists@gmail.com">dbn.lists@gmail.com</a>>