[Mesa-dev] [PATCH 1/3] configure.ac: search for and set PYTHON3

[Jonathan Gray]

On Fri, Apr 22, 2016 at 11:09:52AM -0700, Kenneth Graunke wrote:
> On Saturday, April 23, 2016 1:21:38 AM PDT Jonathan Gray wrote:
> > src/intel/genxml/gen_pack_header.py requires python3.
> > 
> > Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
> > ---
> >  configure.ac | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 6b00cf9..9459489 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -99,6 +99,7 @@ AM_PROG_CC_C_O
> >  AM_PROG_AS
> >  AX_CHECK_GNU_MAKE
> >  AC_CHECK_PROGS([PYTHON2], [python2.7 python2 python])
> > +AC_CHECK_PROGS([PYTHON3], [python3.4 python3])
> >  AC_PROG_SED
> >  AC_PROG_MKDIR_P
> 
> Why mention python3.4 explicitly?  Are we trying to check for 3.4+?
> 
> My system has /usr/bin/python3 -> /usr/bin/python3.5, FWIW...no 3.4.

Yes, I realised that after sending it out and added 3.5

The default python3 version here is python3.4 with python3.5 also
available.  There are no 'python' or 'python3' symlinks.

commit 9b8a12fea59a6fa81b190686aef5298ba3b6b4f8
Author: Jonathan Gray <jsg at jsg.id.au>
Date:   Fri Apr 22 15:04:47 2016 +1000

    configure.ac: search for and set PYTHON3
    
    src/intel/genxml/gen_pack_header.py requires python3.
    
    v2: check for python3.5 as well
    
    Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

diff --git a/configure.ac b/configure.ac
index 6b00cf9..2474078 100644
--- a/configure.ac
+++ b/configure.ac
@@ -99,6 +99,7 @@ AM_PROG_CC_C_O
 AM_PROG_AS
 AX_CHECK_GNU_MAKE
 AC_CHECK_PROGS([PYTHON2], [python2.7 python2 python])
+AC_CHECK_PROGS([PYTHON3], [python3.5 python3.4 python3])
 AC_PROG_SED
 AC_PROG_MKDIR_P
 
@@ -2788,6 +2789,7 @@ if test "x$MESA_LLVM" = x1; then
     echo ""
 fi
 echo "        PYTHON2:         $PYTHON2"
+echo "        PYTHON3:         $PYTHON3"
 
 echo ""
 echo "        Run '${MAKE-make}' to build Mesa"