-
-
Notifications
You must be signed in to change notification settings - Fork 530
Project dependencies are not installed when usedevelop=False #197
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
Comments
Original comment by @DXist For now I just toggle |
Original comment by @DXist I'd like to have ini option. |
Original comment by @hpk42 maybe, would you rather have an ini or command line opt? |
Original comment by @DXist In my case adding '-r' option makes the build 5-6 times longer - that's why I prefer not to set it. When environment is already prepared pip dependency checks are quite fast and introduce much smaller time overhead than full env recreation. I think it would be nice to make "--no-deps" option presence configurable. |
Original comment by @hpk42 speed for the common case. after all, typing "-r" to force recreation is a bit annoying but not super bad IMHO :) |
Original comment by @DXist Could you explain the reason for this? |
Original comment by @hpk42 it does use it but with "--no-deps". |
Original comment by @DXist Thank you for the links! Why does not tox run |
Original comment by @hpk42 yes, it's a known issue (although i can't find an issue number currently) that tox does not know what setup.py contains in terms of dependencies. We could maybe look into the sdist/package's requirements and see if that changed, however, not sure. There are some issues like issue #13 and issue #149 which are similar. |
Original comment by @DXist I activate |
Original comment by @hpk42 tox presumes it controls the virtual environments and that you don't do manual changes in between. How comes you work on the |
I run tox without -r option. At he moment I have to manually install project dependencies when they are changed.
Steps I do to reproduce this:
When I set
usedevelop=True
the problem disappears.The text was updated successfully, but these errors were encountered: