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

[Matt Turner]

On Mon, Mar 4, 2013 at 11:14 AM, Brian Paul <brianp at vmware.com> wrote:
> 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?

Sure.

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

Thanks!