-
-
Notifications
You must be signed in to change notification settings - Fork 615
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
Sequential pip-compile calls leads to missed dependecies #1154
Comments
Hello @pohmelie, Thanks for the report. Couldn't reproduce the bug. See details below: $ cat foo/setup.py
from setuptools import setup
setup(name='foo', version='0.1', install_requires=['pytz'])
$ cat bar/setup.py
from setuptools import setup
setup(name='bar', version='0.1', install_requires=['six'])
$ cat requirements.in
file:./foo
click
$ pip-compile --no-header
file:./bar # via -r requirements.in
click==7.1.2 # via -r requirements.in
file:./foo # via -r requirements.in
pytz==2020.1 # via foo
six==1.15.0 # via bar
$ cp requirements.txt requirements.txt.orig
$ pip-compile --no-header
file:./bar # via -r requirements.in
click==7.1.2 # via -r requirements.in
file:./foo # via -r requirements.in
pytz==2020.1 # via foo
six==1.15.0 # via bar
$ sha1sum requirements.txt*
ac59f831da785a68612372a69d633742eb356085 requirements.txt
ac59f831da785a68612372a69d633742eb356085 requirements.txt.orig |
Maybe, it depends that origin setup.py have local reqs:
Or is it better to create github repo to repoduce this thing? |
That'd be super helpful! |
Also, could you reproduce the bug on |
Ok, here is our exact case skeleton: https://github.com/pohmelie/pip-tools-1154 |
|
Looking at the changes in Maybe the 5.1.0 output is a hint to what is happening: pip-compile can't find |
Thanks a lot, that's perfect! Will look into that. |
Looks like it is a regression that was introduced in c0b33e7 (#1132). There is also a related issue #1155. /cc @richafrank |
So this is the real bug we have to figure out. |
The workaround is to add an egg fragment to ...
file:./krontech-core#egg=krontech-core
... UPDATE: and temporarily downgrade to |
Okay, this is not critical for us, so watching this issue 👀 |
|
We have subnormal requirements.in:
So, first two lines links to submodules with simple setup.py package description.
Environment Versions
Steps to replicate
pip-compile requirements.in -o requirements.txt
pip-compile requirements.in -o requirements.txt
Expected result
They are the same
Actual result
Packages from submodules
krontech-core
andkrontech-web
are missed.The text was updated successfully, but these errors were encountered: