Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

implement pep-0394 #8

Merged
merged 1 commit into from Feb 12, 2015
Merged

Conversation

Dieterbe
Copy link
Contributor

make it explicit that codemod needs python2,
as it does not run with python3.
according to pep-0394, python distributors are allowed to provide either
python 2 or 3 as python command, and should provide python2 and
python3 commands to refer to python v2 or 3 respectively.

see http://www.python.org/dev/peps/pep-0394/

without this fix, codemod doesn't run on platforms where python is
python3, such as Arch Linux

make it explicit that codemod needs python2,
as it does not run with python3.
according to pep-0394, python distributors are allowed to provide either
python 2 or 3 as `python` command, and should provide `python2` and
`python3` commands to refer to python v2 or 3 respectively.

see http://www.python.org/dev/peps/pep-0394/

without this fix, codemod doesn't run on platforms where `python` is
python3, such as Arch Linux
@ptarjan
Copy link
Contributor

ptarjan commented Sep 18, 2012

How hard is it to make it work with python 3?

@Dieterbe
Copy link
Contributor Author

not sure, I have no interest in porting to python3 right now.

@Dieterbe
Copy link
Contributor Author

can this be merged? it's still broken.

@modocache
Copy link
Contributor

Woah, @Dieterbe, sorry for the slow turnaround on this! 🙊 💦

This looks like a great change, thanks!

modocache added a commit that referenced this pull request Feb 12, 2015
@modocache modocache merged commit fbe304b into facebookarchive:master Feb 12, 2015
@Dieterbe
Copy link
Contributor Author

blast from the past!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants