Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Better explain in Docker docs gotcha with platforms and pex_binary #16393

Closed
Eric-Arellano opened this issue Aug 3, 2022 · 0 comments · Fixed by #17096
Closed

Better explain in Docker docs gotcha with platforms and pex_binary #16393

Eric-Arellano opened this issue Aug 3, 2022 · 0 comments · Fixed by #17096
Labels
backend: Docker Docker backend-related issues backend: Python Python backend-related issues documentation

Comments

@Eric-Arellano
Copy link
Contributor

Okay I think what is happening is that for now, Pants creates the binary on your mac beforehand and then copies it into the container. It does not create it in the container (although that is highly requested and we want to do it)

So, the binary is being built w/ macOS deps, and failing to run in Linux. The solution is to use platforms or complete_platforms

This is not very intuitive. While #13682 would be better, in the meantime, this should be better documented.
@Eric-Arellano Eric-Arellano changed the title Better explain in Docker docks gotcha with platforms and pex_binary Better explain in Docker docs gotcha with platforms and pex_binary Aug 3, 2022
@thejcannon thejcannon added backend: Python Python backend-related issues backend: Docker Docker backend-related issues labels Aug 23, 2022
@stuhood stuhood mentioned this issue Oct 4, 2022
Merged
stuhood added a commit that referenced this issue Oct 28, 2022
@stuhood
Initial documentation for Environments (#17096)
d7880ce 
Adds initial documentation for the Environments feature designed and implemented in #7735 and #13682.

This initial version is meant to capture what will be possible in the `2.15.x` release, but it will/should contain some references to future work (some of which should block stabilizing environments).

Fixes #13682, and fixes #16393.

[ci skip-rust]
[ci skip-build-wheels]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend: Docker Docker backend-related issues backend: Python Python backend-related issues documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Initial documentation for Environments stuhood/pants
2 participants
@thejcannon @Eric-Arellano