Skip to content

Switch to python 3. #17

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

Merged
merged 1 commit into from
Sep 28, 2016
Merged

Conversation

petterreinholdtsen
Copy link
Contributor

No description provided.

@aclements
Copy link
Owner

Hi @petterreinholdtsen. What's the motivation for doing this? Your commit message didn't say.

@petterreinholdtsen
Copy link
Contributor Author

[Austin Clements]

Hi @petterreinholdtsen. What's the motivation for doing this? Your
commit message didn't say.

My motivation is to make sure the build work independent of python
version, to make sure the debian package can build depend on python
without a specific version number. Without the patch, the package build
fail to work with python >= 3.

Happy hacking
Petter Reinholdtsen

@petterreinholdtsen
Copy link
Contributor Author

Would you prefer to just call 'python' instead of replacing python2 with python3?

@aclements
Copy link
Owner

My motivation is to make sure the build work independent of python
version, to make sure the debian package can build depend on python
without a specific version number.

I'll go ahead and submit this because Python 3 is the future and all (to be clear, I actually quite like Python 3, I just don't think stable things should be translated without good reason since the translation may introduce bugs).

However, I'm afraid I still don't understand the motivation. This change does not make the build work independent of Python version; with this change it only works with Python 3. Also, my understanding of the policy is that the "python" package will continue to refer to Python 2, so it would be fine to declare a build dependency on the "python" package, just like how with this change it will need to depend on "python3".

@aclements aclements merged commit 7c62606 into aclements:master Sep 28, 2016
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.

2 participants