On 15 August 2018 at 17:18, Dylan Baker <dy...@pnwbakers.com> wrote:
> We don't want to support older versions of python 2 anymore, and we
> don't support python 3.x in autotools currently.
> ---
>  configure.ac | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index c2155a541b0..78672734d06 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -162,6 +162,13 @@ if test -z "$PYTHON2"; then
>          AC_MSG_ERROR([Python not found - unable to generate sources])
>      fi
>  else
> +    PYTHON_VERSION=`$PYTHON2 --version |& awk '{print $2}'`  dnl Yes, 
> python2 prints it's version to stderr
> +    PYTHON_MAJOR=`echo $PYTHON_VERSION | cut -d . -f 1`
> +    PYTHON_MINOR=`echo $PYTHON_VERSION | cut -d . -f 2`
> +    if test $PYTHON_MAJOR -ne 2 -o $PYTHON_MINOR -ne 7; then
> +        AC_MSG_ERROR([Python version 2.7.x not found - unable to generate 
> sources])
> +    fi
> +
A slightly better alternative is to use AM_PATH_PYTHON as seen here [1]
Since there's no reason to keep autoconf python2 only, I've prepped a
small series [2].

Once everything is green I'll send it out. Until then, feel free to
drop this or reuse as much/little from my patch.

Thanks
Emil

[1] 
https://github.com/evelikov/Mesa/commit/342060879b57b53cbd034c5afac51d71b8b414e7
[2] https://github.com/evelikov/Mesa/commits/autoconf-python3
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to