-
-
Notifications
You must be signed in to change notification settings - Fork 611
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
platform_system
not correctly respected
#1793
Comments
Not “the first”, just “current env”. See https://pip-tools.readthedocs.io/en/latest/#cross-environment-usage-of-requirements-in-requirements-txt-and-pip-compile and #826 (comment). |
Thanks, that's very helpful. I didn't realise it was because I was running on Mac and also that the feature I thought existed does not exist (yet). Will close this in favour of #826 then |
Is there any feature planned to support cross-platform builds? For example, my team is lucky enough to have M1 Macs to develop on, but for unhappy reasons beyond our control we are stuck deploying to x86-64. We've been OK for now just running As far as I understand, the pedantically-correct alternative would be to run It would be great if I could run something like |
@gwerbin I don't think it's possible to implement with the current state of the ecosystem. pip-tools doesn't do anything special around building individual dists — it just uses pip and whatever underlying tooling each dist calls. It's up to you to provide an environment that is actually capable of building those things. |
pip-compile
does not respectplatform_system
correctly when it is specified multiple times for the same package. This is useful in cases where you need to have different versions of the same package on different OSs.Environment Versions
Steps to replicate
Run
pip-compile
on the followingrequirements.in
file:Expected result
grpcio-tools
should occur twice in therequirements.txt
in a similar way to how it was specifiedActual result
Instead you get something like this where the dependency we specified only occurs once with the first platform_system.
The text was updated successfully, but these errors were encountered: