[Mesa-dev] [PATCH v2 3/5] configure: Enforce python 2.7.x

Emil Velikov emil.l.velikov at gmail.com
Thu Aug 16 13:27:08 UTC 2018


On 15 August 2018 at 17:18, Dylan Baker <dylan at 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


More information about the mesa-dev mailing list