Makefile: honor PYTHON configuration properly
authorClément Bœsch <u@pkh.me>
Mon, 14 Aug 2017 06:59:11 +0000 (08:59 +0200)
committerTom Rini <trini@konsulko.com>
Sun, 20 Aug 2017 13:53:13 +0000 (09:53 -0400)
commit3809e30273e03d762595dbc2a62f3a8398281ec8
treece5511e621b0fa490935dfb8506e3c23398e2830
parent2629a21e209d91cdb778f43612235ed1f3029488
Makefile: honor PYTHON configuration properly

On some systems `python` is `python3` (for instance, Archlinux). The
`PYTHON` variable can be used to point to `python2` to have a successful
build.

The use of `PYTHON` is currently limited in the Makefile and needs to be
extended in other places:

First, pylibfdt is required to be a Python 2 binding (binman imports
pylibfdt and is only compatible Python 2), so its setup.py needs to be
called accordingly. An alternative would be to change the libfdt
setup.py shebang to python2, but the binding is actually portable. Also,
it would break on system where there is no such thing as `python2`.

Secondly, the libfdt import checks need to be done against Python 2 as
well since the Python 2 compiled modules (in this case _libdft.so) can
not be imported from Python 3.

Note on the libfdt imports: "@if ! PYTHONPATH=tools $(PYTHON) -c 'import
libfdt'; then..." is probably simpler than the currently sub-optimal
pipe.
Reviewed-by: Jonathan Gray <jsg@jsg.id.au>
Makefile
scripts/Makefile.spl
tools/Makefile