-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Pipfile.lock doesn't preserve some extras
information
#1014
Comments
Thanks for the full report @grets, that was really helpful 😄 FYI, this is not related to #1013, even if it might look like it. Good thinking, but those a really distinct issues, I can assure it. Now, for this issue: And I'm pretty sure this is exactly the issue here, here what happens:
So yeah, that's the known issue of "wrong python version used for locking" we've been hunting for a while. One additional thing I would ask, to get rid of that 0.1% of doubt I have about the cause of the issue: With the exact same setup, could you redo all of this, and at the end, do |
Thanks for the response @vphilippon, I went ahead and ran the
Pipfile.lock generated from the same Pipfile as in my first comment but using pipenv with python2.7 this time and everything is as expected
|
Also, the Pipfile.lock generated using |
@macbeth322 Allright, that confirms it, thanks! |
I believe I stumbled upon a similar issue. I'm trying to deploy my app to docker with pipenv installing requirements. I added an extras package yesterday and now none of the dependencies to airflow exist in my lock file (notably
I can send you my Pipfile, Pipfile.lock, and/or other files if they would be relevant to the issue at hand. |
@sethwoodworth I only really have one question about this -- in the root directory of your project, do you have a subdirectory called |
@techalchemy Yes! I have precisely that. A folder named |
Nailed it! Seen this one enough by now to know right away what’s up there |
This is fixed in #958 |
#958 fixes the relative path issue, but does it also fix the correct python version being used in lock?
|
This is a dupe of #857, though. |
@helderco the local file thing and the python version thing are separate. The former is fixed, but dependency resolution with regard to the latter isn’t fixed yet |
I also filed a possibly related issue here https://github.com/kennethreitz/pipenv/issues/1013
Environment
Expected result
Should be able to reinstall
extra
dependencies from a Pipfile.lockIn this example
raven[flask]
in python 2 depends onBlinker
andFlask
because of theflask
extra, and depends oncontextlib2
because its python < 3.4; so these dependencies should be installed.https://github.com/getsentry/raven-python/blob/master/setup.py
Actual result
pipenv install raven[flask]
works as expected but after recreating the virtual environment, installing from Pipfile.lock doesn't install all extra dependencies and some extra dependencies are absent from the Pipfile.lock.Steps to replicate
Showing
pipenv install raven[flask]
installs all expected packagesShowing
pipenv install
doesn't install some extra dependenciesShowing
pipenv install --skip-lock
installs expected dependencies butpipinstall upgrade
uninstalls some.Pipfile
Pipfile.lock
The text was updated successfully, but these errors were encountered: