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

Add pipenv caveat about mixing Python versions #427

Closed
ncoghlan opened this issue Feb 2, 2018 · 3 comments · Fixed by #430
Closed

Add pipenv caveat about mixing Python versions #427

ncoghlan opened this issue Feb 2, 2018 · 3 comments · Fixed by #430
Assignees
Labels
type: bug A confirmed bug or unintended behavior

Comments

@ncoghlan
Copy link
Member

ncoghlan commented Feb 2, 2018

Due to pypa/pipenv#857, pipenv can currently get environment marker evaluation wrong when generating the lock file if the target venv uses a version of Python other than the one running pipenv.

This is most frequently a problem when setting up a Py2 venv from Py3 (due to the prevalence of Py2 only conditional dependencies), but issues can also arise when creating venvs for older Python 3 versions.

@theacodes
Copy link
Member

should we just wait for the upstream issue to be addressed or do we anticipate this guide needing this caveat long-term?

@ncoghlan
Copy link
Member Author

ncoghlan commented Feb 5, 2018

I think it's worth adding the caveat in the near term, as the pipenv folks have been trying different ways of fixing this over the past few releases, and it isn't clear yet when a more robust approach to mixing and matching Python versions will ship.

As things currently stand, we're seeing folks trying to use a recent Python 3 to manage Python 2.7 projects (rather than developing & deploying on the same Python version), and wondering why their Python-2-only dependencies aren't showing up properly (and getting an overly negative impression of pipenv as a result).

Even after it's fixed, we may want to keep the caveat in some form, just noting the minimum version needed for reliable cross-version operation.

@theacodes theacodes added type: bug A confirmed bug or unintended behavior and removed state: needs clarification labels Feb 5, 2018
@theacodes
Copy link
Member

Sounds good. I'll try to get this up this week, but if someone wants to send a PR to beat me to it I'll merge quickly. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A confirmed bug or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants