[Mesa-dev] [PATCH 1/2] configure.ac: Don't check for X11 unconditionally.

[Brian Paul]

On 03/04/2013 11:31 AM, Matt Turner wrote:
> X11 is already checked conditionally below.
>
> Fixes OSMesa-only configurations to not require X11.
> ---
>   configure.ac |    8 --------
>   1 files changed, 0 insertions(+), 8 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 7852595..1b13d06 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -832,20 +832,12 @@ if test "x$enable_dri" = xyes; then
>       fi
>   fi
>
> -dnl Find out if X is available.
> -PKG_CHECK_MODULES([X11], [x11], [no_x=no], [no_x=yes])
> -
>   dnl Try to tell the user that the --x-* options are only used when
>   dnl pkg-config is not available. This must be right after AC_PATH_XTRA.
>   m4_divert_once([HELP_BEGIN],
>   [These options are only used when the X libraries cannot be found by the
>   pkg-config utility.])
>
> -dnl We need X for xlib and dri, so bomb now if it's not found
> -if test "x$enable_glx" = xyes -a "x$no_x" = xyes; then
> -    AC_MSG_ERROR([X11 development libraries needed for GLX])
> -fi
> -
>   dnl Direct rendering or just indirect rendering
>   case "$host_os" in
>   gnu*)

The series LGMT.  Thanks, Matt!

Candidates for the stable branches?

Reviewed-by: Brian Paul <brianp at vmware.com>