[Mesa-dev] [PATCH v2] configure.ac: Disable GLX if OpenGL is not enabled

[Ian Romanick]

On 01/31/2013 08:50 AM, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
>
> GLX uses mapi/glapi/libglapi.la, which is only built for OpenGL.
>
> NOTE: This is a candidate for the stable branches.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59364
>
> Tested-by: Tom Stellard <thomas.stellard at amd.com>
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>

Technically, you could create an ES2 context on GLX using 
GLX_EXT_create_context_es2_profile, but meh.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> ---
>
> v2: Error out on --enable-xlib-glx --disable-opengl, which is
>      self-contradictory.
>
>   configure.ac |   10 ++++++++++
>   1 file changed, 10 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index cfd52bf..95c9f3d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -701,6 +701,16 @@ if test "x$enable_dri$enable_xlib_glx" = xyesyes; then
>       AC_MSG_ERROR([DRI and Xlib-GLX cannot be built together])
>   fi
>
> +if test "x$enable_opengl$enable_xlib_glx" = xnoyes; then
> +    AC_MSG_ERROR([Xlib-GLX cannot be built without OpenGL])
> +fi
> +
> +# Disable GLX if OpenGL is not enabled
> +if test "x$enable_glx$enable_opengl" = xyesno; then
> +    AC_MSG_WARN([OpenGL not enabled, disabling GLX])
> +    enable_glx=no
> +fi
> +
>   # Disable GLX if DRI and Xlib-GLX are not enabled
>   if test "x$enable_glx" = xyes -a \
>           "x$enable_dri" = xno -a \
>