Commit a280b53d authored by Kirill Smelkov's avatar Kirill Smelkov

Fix build with non-std "python"

In 5755a6b3 (Basic setup.py / Makefile to build/install/sdist stuff +
bigfile.so skeleton) we overlooked one thing:

    when tailing from setup.py to make level, we forgot to setup the
    PYTHON env variable properly, and that leads to changing building
    with std python.

Look e.g.

    $ python3 setup.py build_ext -i
    running build_ext
    python setup.py ll_build_ext --inplace
    ^^^

which is obviously not correct.

Fix it.

The tox setup we have setup for testing with multiple interpreters
(7af2b2d7 "Tox setup to test things with py27 & py34 and several
versions of ZODB") did not caught this, because for every python and
environment versions tox initializes separate virtualenv, in which
.../bin/python is a link to chosen python, and this way both python and
my-selected-for-build-python are the same.

Cc: @Tyagov
parent c068e6b1
...@@ -22,6 +22,7 @@ from distutils.errors import DistutilsExecError ...@@ -22,6 +22,7 @@ from distutils.errors import DistutilsExecError
from subprocess import check_output as runcmd from subprocess import check_output as runcmd
import os import os
import sys
_bigfile = Extension('wendelin.bigfile._bigfile', _bigfile = Extension('wendelin.bigfile._bigfile',
...@@ -111,7 +112,7 @@ class build_py(_build_py): ...@@ -111,7 +112,7 @@ class build_py(_build_py):
# run `make <target>` # run `make <target>`
def runmake(target): def runmake(target):
err = os.system('make %s' % target) err = os.system('make %s PYTHON="%s"' % (target, sys.executable))
if err: if err:
raise DistutilsExecError('Failed to execute `make %s`' % target) raise DistutilsExecError('Failed to execute `make %s`' % target)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment