build fails when python3 is default python

[donpdonp]

1. python configure script crashes on python3

/tmp/iojs-v1.0.0 $ python -V
Python 3.3.3
/tmp/iojs-v1.0.0 $ ./configure --help
  File "./configure", line 332
    '''
      ^
SyntaxError: invalid syntax

1. getting around deps: update openssl to 1.0.1j #1 with "$python2 configure", the final step crashes which also looks to be python3 related.

make[1]: Entering directory `/tmp/iojs-v1.0.0/out'
  LD_LIBRARY_PATH=/tmp/iojs-v1.0.0/out/Release/lib.host:/tmp/iojs-v1.0.0/out/Release/lib.target
:$LD_LIBRARY_PATH; export LD_LIBRARY_PATH; cd ../deps/v8/tools/gyp; mkdir -p /tmp/iojs-v1.0.0/o
ut/Release/obj/gen; python ../../tools/js2c.py "/tmp/iojs-v1.0.0/out/Release/obj/gen/libraries.
cc" CORE ../../src/runtime.js ../../src/v8natives.js ../../src/symbol.js ../../src/array.js ../
../src/string.js ../../src/uri.js ../../src/third_party/fdlibm/fdlibm.js ../../src/math.js ../.
./src/apinatives.js ../../src/date.js ../../src/regexp.js ../../src/arraybuffer.js ../../src/ty
pedarray.js ../../src/generator.js ../../src/object-observe.js ../../src/collection.js ../../sr
c/weak-collection.js ../../src/collection-iterator.js ../../src/promise.js ../../src/messages.j
s ../../src/json.js ../../src/array-iterator.js ../../src/string-iterator.js ../../src/debug-de
bugger.js ../../src/mirror-debugger.js ../../src/liveedit-debugger.js ../../src/macros.py
Traceback (most recent call last):
  File "../../tools/js2c.py", line 534, in <module>
    main()
  File "../../tools/js2c.py", line 530, in main
    JS2C(args[2:], args[0], args[1], options.raw, options.startup_blob)
  File "../../tools/js2c.py", line 499, in JS2C
    sources = PrepareSources(source)
  File "../../tools/js2c.py", line 370, in PrepareSources
    assert len(macro_files) in [0, 1]
TypeError: object of type 'filter' has no len()
make[1]: *** [/tmp/iojs-v1.0.0/out/Release/obj/gen/libraries.cc] Error 1
make[1]: Leaving directory `/tmp/iojs-v1.0.0/out'

[bnoordhuis]

Try make PYTHON=python2.

[myf]

this make PYTHON=python2 would fix the ./configure but it will still break the make

there are hardcoded python in v8 that does not honor this $PYTHON environment variable.

the ArchLinux AUR package basically swapped the header of all the .py files in this fashion:

prepare() {
  cd "${srcdir}/${pkgname}"

  find -type f -exec sed \
    -e 's_^#!/usr/bin/env python$_&2_' \
    -e 's_^\(#!/usr/bin/python2\).[45]$_\1_' \
    -e 's_^#!/usr/bin/python$_&2_' \
    -e "s_'python'_'python2'_" -i {} \;
  find test/ -type f -exec sed 's_python _python2 _' -i {} \;
}

Wonder if there's easier way to do this?

[jbergstroem]

Something similar would have to be done on for instance FreeBSD. This has been brought up previously in the nodejs issue trackers where the conclusion was that maintaining a symlink for the user (python -> python27) was the simplest solution. I'm not super happy with that either and would like some simple detection.

I don't think python3 will be supported until gyp is out of the picture. The effort for supporting python3 would be a pretty trivial task from there.

[rvagg]

on a couple of the build machines we take the easy path and symlink python2 to ~/bin/python and run make with PATH=~/bin/:${PATH} make ...

[jbergstroem]

Another way could be changing configure from a python shebang to a shell shebang, do some simple checks for which python binary we'd like to use, then pass the rest of the configure to $PYTHON -c. It requires some minor rewriting of configure (__filename__ for instance), but it'd work. Thoughts?

[rvagg]

the problem is deeper, python is hardwired into a bunch of v8 build components

[bjconlan]

Yeah, I thought 2.7.x series was a requirement (as it is for gyp/node-gyp) I wouldn't be surprised if this is closed (wontfix) by an authority.

[Fishrock123]

Is this a v8 issue and not fixable from our end?

[bnoordhuis]

Yes, both a V8 and a gyp issue. Not much we can do about that. Closing, sorry.