Code review comment for ~aixtools/cloud-init:hash-bang

Revision history for this message
Scott Moser (smoser) wrote :

the one thing that sucks about this is that virtualenv tries to help by making 'python3' have a name 'python'.

$ tox-venv py3 python --version
Python 3.5.3rc1

the change you've suggested here is probably right, but i really hate that it opens us up to getting the wrong python. (Ie, my feeling is that /usr/bin/python is and always will be python 2, where '/usr/bin/env python' might be python3).

most of that is rant, but the short result is that you now have to verify that the things you've changed from 'python' (python2) to 'env python' (python2 or python3) are python3 compatible.

Can you verify that ?

« Back to merge proposal