Bug 96176

Summary:	Cannot build non-intel drivers without python3.
Product:	Mesa	Reporter:	Vinson Lee <vlee>
Component:	Mesa core	Assignee:	mesa-dev
Status:	RESOLVED FIXED	QA Contact:	mesa-dev
Severity:	normal
Priority:	medium	CC:	emil.l.velikov, joro-2013
Version:	git	Keywords:	bisected, regression
Hardware:	x86-64 (AMD64)
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Vinson Lee 2016-05-25 06:17:23 UTC

mesa: 998829f4045a98d16797e9d2f5094c35eb272909 (master 11.3.0-devel)

Regression introduced with this commit.

commit 2cd687ce97f39bd98fab4d72d607b7a1644cd2a0
Author: Emil Velikov <emil.velikov@collabora.com>
Date:   Wed May 4 11:47:14 2016 +0100

    configure.ac: error out when building from git without python3
    
    Bail early, as opposed to later on during the build.
    
    Signed-off-by: Emil Velikov <emil.velikov@collabora.com>

Comment 1 Emil Velikov 2016-05-25 12:56:13 UTC

I wouldn't call it regression but intentional change ;-)

The long term goal is to port the python2 scripts to python3. Thus adding a list of generated sources and/or other heuristics in configure is fragile and not going to scale.

Is python3 not available on your system, which distribution are you using here ?

Comment 2 Vinson Lee 2016-05-25 18:59:57 UTC

Mac OS X and CentOS do not have Python 3.

Comment 3 Vinson Lee 2016-05-28 18:39:10 UTC

*** Bug 96242 has been marked as a duplicate of this bug. ***

Comment 4 Emil Velikov 2016-06-01 15:09:44 UTC

The script that inspired this change was made python2 compatible and the requirement was removed with the series ending

commit 604010a7edbe03ff65720cab8dddba1d0ca1571b
Author: Dylan Baker <dylan@pnwbakers.com>
Date:   Tue May 31 13:31:44 2016 -0700

    Don't use python 3
    
    Now there are not files that require python 3, so for now just remove
    the python 3 dependency and use python 2. I think the right plan is to
    just get all of the python ready for python 3, and then use whatever
    python is available.
    
    Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
    Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
    cc: 12.0 <mesa-stable@lists.freedesktop.org>

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.