-
Notifications
You must be signed in to change notification settings - Fork 3k
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
pip 20.3b1 fails to reinstall packages in -e .
mode (despite changes)
#9116
Comments
Maybe related to #8711? |
Indeed, this is probably just one aspect of what is discussed there. We may close this as a duplicate. I had searched for existing issues but since the title of the other issue doesn't really describe the problem I couldn't find anything. |
Is "local development" not the same as "editable"? Since #8711 misses a concrete issue description I assumed it refers to |
Not necessarily. Making changes and then reinstalling, and never using |
FWIW, this is one my things-to-do-before-release list, as noted in #9147 (comment). |
Environment
Description
Starting with version 20.3b1,
pip install -e .
fails to re-install a package even though it has changes that should lead to a re-installation.Expected behavior
Automatic re-installation was the standard in earlier pip versions and may be the preferred/expected behavior?
How to Reproduce
Consider this minimal example structure:
setup.py
setup.cfg
foobar/foo1.py
Now running
pip install -e . -vvv
for the first time leads to:stdout
The entrypoint
foo1
is available at that point. So far so good.Now duplicating
foobar/foo1.py
intofoobar/foo2.py
, adding the corresponding entry pointfoo2 = foobar.foo2:main
, and rerunningpip install -e . -vvv
results in:stdout
As a user I would expect entrypoint
foo2
to be available now as well, but it isn't.In previous pip versions (verified against 20.2.4) this kind of
pip install -e .
workflow for local packages without explicit versioning was working fine.The obvious work-around is to use
--force
, but the question is if skipping installs is really intended.Output
See inline above.
The text was updated successfully, but these errors were encountered: