Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#27878 closed Bug (invalid)

Move to Python 3 shebangs

Reported by: Chris Lamb Owned by: nobody
Component: Core (Other) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Now that master branch is Python 3 only, we should use Python 3 shebangs. I just ran runtests.py and it used Python 2.. :)

Change History (8)

comment:1 by Chris Lamb, 7 years ago

comment:2 by Tim Graham <timograham@…>, 7 years ago

Resolution: fixed
Status: newclosed

In 4cffa9a1:

Fixed #27878, refs #23919 -- Used python3 shebangs.

comment:3 by Tim Graham, 7 years ago

Component: UncategorizedCore (Other)

I found that the Windows installation of Python 3.x does not have an executable called python3, so I'm not sure if we should keep this change.

$ ./tests/runtests.py settings_tests
/usr/bin/env: ‘python3’: No such file or directory

comment:4 by Claude Paroz, 7 years ago

I remember about discussions on this issue. Different systems/distributions have different strategies around using python, python2 or python3.
I would tend to think that the environment in which you are running your Python scripts should point python to the right version. So +1 for reverting.

comment:5 by Tim Graham <timograham@…>, 7 years ago

In 784a53b:

Reverted "Fixed #27878, refs #23919 -- Used python3 shebangs."

This reverts commit 4cffa9a1ffb37d4de7e99a851a9ed87b3c02d229.

comment:6 by Tim Graham, 7 years ago

Resolution: fixedinvalid

comment:7 by Chris Lamb, 7 years ago

the environment in which you are running your Python scripts should point python to the right version

Hm, except for legacy reasons it is not really possible to change (eg. all Debian-derived distributions).

comment:8 by Aymeric Augustin, 7 years ago

pyenv helps -- it's the most convenient solution I know for running a modern Python on Debian.

Note: See TracTickets for help on using tickets.
Back to Top