astroidmail / homebrew-astroid

Homebrew tap for Astroid
10 stars 5 forks source link

scons needs python 2 #9

Closed gauteh closed 7 years ago

gauteh commented 7 years ago

got this issue reported:

screen shot 2017-05-13 at 10 27 01

older python 2 is already installed somewhere

cpiro commented 7 years ago

I'm not sure this a problem with astroid's formula. We use the scons method, which calls the executable installed by the scons formula. 1/ It's up to the scons formula to make that executable work correctly 2/ its shebang (on my system at least) is /usr/bin/env python, so the interpreter is whatever python executable is handy, which should really be Python 2. macOS ships with python meaning Python 2, and I'm pretty sure Homebrew is careful to install Python 3 as python3. If I the bug reporter runs python --version, does it return Python 3.5.2?


$ cw /usr/local/opt/scons/bin/scons
/usr/local/opt/scons/bin/scons is /usr/local/opt/scons/bin/scons
/usr/local/Cellar/scons/2.5.1/libexec/scons.py
/usr/local/opt/scons/bin/scons: symbolic link to ../libexec/scons.py
/usr/local/Cellar/scons/2.5.1/libexec/scons.py: a /usr/bin/env python script text executable, ASCII text
#! /usr/bin/env python
#
# SCons - a Software Constructor
...
gauteh commented 7 years ago

Ah right, I thought it might needed a python2 scons somewhere.. maybe python was not installed via brew, but conda or something.

@patricknraanes: ping, if you have time to debug

patnr commented 7 years ago

Indeed, my default python comes from anaconda and is version 3.5. (Don't have much time today, but perhaps tomorrow)

gauteh commented 7 years ago

Closing, untill we get more info or the issue occurs again.