Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforces Python2 usage #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Enforces Python2 usage #39

wants to merge 1 commit into from

Conversation

HalisCz
Copy link

@HalisCz HalisCz commented Dec 29, 2015

Should fix #38

@nickjj
Copy link
Owner

nickjj commented Dec 30, 2015

Isn't it idiomatic Python to assume python is 2 where as python3 is specifically 3?

In this case, the problem is that Ansigenome doesn't support 3 at the moment but I'm not sure I agree with forcing 2 in the mean time. A more long term approach would be to get Ansigenome running on both 2 and 3. Problem is, I don't have 3 installed atm.

@HalisCz
Copy link
Author

HalisCz commented Dec 30, 2015

Isn't it idiomatic Python to assume python is 2 where as python3 is specifically 3?

It should be, according to PEP 394, but as PEP 394 says, on Arch Linux is /usr/bin/python linked to /usr/bin/python3
I made some checks across distributions and this is the compatibility of the ansigenome v.0.5.6:

  • Arch - /usr/bin/python points to python3
  • Centos 7.2 - /usr/bin/python points to python2
  • Debian 8 - /usr/bin/python points to python2
  • Fedora 23 - in minimal image is python3 installed, python doesn't exists
  • Gentoo - /usr/bin/python points to python3
  • Ubuntu 14.04 - in minimal image is python3 installed, python doesn't exists

In this case, the problem is that Ansigenome doesn't support 3 at the moment but I'm not sure I agree with forcing 2 in the mean time. A more long term approach would be to get Ansigenome running on both 2 and 3. Problem is, I don't have 3 installed atm.

I disagree. The problem I've encountered is that my system (Gentoo) links python to python3, but contains python2 too. So with this pull request I am able to run ansigenome.

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

Successfully merging this pull request may close these issues.

Python 3 incompatibility
2 participants