[U-Boot] [PATCH] pylibfdt: Use Python 2 in Makefile

pylibfdt needs Python 2 to build. Replace $(PYTHON) with $(PYTHON2) in pylibfdt Makefile to ensure Python 2 is used to build it.
This fixes build on systems where Python 3 is the default version of the "python" interpreter. --- scripts/dtc/pylibfdt/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/dtc/pylibfdt/Makefile b/scripts/dtc/pylibfdt/Makefile index c769d7db06..15e66ad44d 100644 --- a/scripts/dtc/pylibfdt/Makefile +++ b/scripts/dtc/pylibfdt/Makefile @@ -21,7 +21,7 @@ quiet_cmd_pymod = PYMOD $@ CPPFLAGS="$(HOSTCFLAGS) -I$(LIBFDT_srcdir)" OBJDIR=$(obj) \ SOURCES="$(PYLIBFDT_srcs)" \ SWIG_OPTS="-I$(LIBFDT_srcdir) -I$(LIBFDT_srcdir)/.." \ - $(PYTHON) $< --quiet build_ext --inplace + $(PYTHON2) $< --quiet build_ext --inplace
$(obj)/_libfdt.so: $(src)/setup.py $(PYLIBFDT_srcs) FORCE $(call if_changed,pymod)

On Tue, 8 Jan 2019 at 06:19, Josef Lusticky josef@lusticky.cz wrote:
pylibfdt needs Python 2 to build. Replace $(PYTHON) with $(PYTHON2) in pylibfdt Makefile to ensure Python 2 is used to build it.
This fixes build on systems where Python 3 is the default version of the "python" interpreter.
scripts/dtc/pylibfdt/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
At some point we should sync this with upstream which I think plans to support both.

On Thu, Jan 10, 2019 at 05:56:56AM -0700, Simon Glass wrote:
On Tue, 8 Jan 2019 at 06:19, Josef Lusticky josef@lusticky.cz wrote:
pylibfdt needs Python 2 to build. Replace $(PYTHON) with $(PYTHON2) in pylibfdt Makefile to ensure Python 2 is used to build it.
This fixes build on systems where Python 3 is the default version of the "python" interpreter.
scripts/dtc/pylibfdt/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
At some point we should sync this with upstream which I think plans to support both.
Given the impending death of Python2, that's good news.

On Tue, Jan 08, 2019 at 02:19:46PM +0100, Josef Lusticky wrote:
pylibfdt needs Python 2 to build. Replace $(PYTHON) with $(PYTHON2) in pylibfdt Makefile to ensure Python 2 is used to build it.
This fixes build on systems where Python 3 is the default version of the "python" interpreter. Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot/master, thanks!
participants (3)
-
Josef Lusticky
-
Simon Glass
-
Tom Rini