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.

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

Add --site-packages-copies for external venvs. #2470

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

jsirois
Copy link
Member

@jsirois jsirois commented Jul 23, 2024

When creating a venv external to the PEX_ROOT via either
pex3 venv create ... or PEX_TOOLS=1 ./my.pex venv ... you can now
specify --site-packages-copies to ensure all code populated in the
venv is isolated from the PEX_ROOT cache. Although this takes more
space on disk, it may make sense to use when you expect the venv might
be tampered with or used for experimentation that might alter its files;
otherwise contaminating the PEX_ROOT cache.

Fixes #2313

When creating a venv external to the `PEX_ROOT` via either
`pex3 venv create ...` or `PEX_TOOLS=1 ./my.pex venv ...` you can now
specify `--site-packages-copies` to ensure all code populated in the
venv is isolated from the `PEX_ROOT` cache. Although this takes more
space on disk, it may make sense to use when you expect the venv might
be tampered with or used for experimentation that might alter its files;
otherwise contaminating the `PEX_ROOT` cache.

Fixes pex-tool#2313
@jsirois jsirois requested review from zmanji, kaos, benjyw and huonw July 23, 2024 22:27
@zmanji
Copy link
Collaborator

zmanji commented Jul 23, 2024

🚀

Copy link
Collaborator

@zmanji zmanji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes and tests lgtm!

@jsirois jsirois merged commit aaa4e43 into pex-tool:main Jul 24, 2024
26 checks passed
@jsirois jsirois deleted the issues/2313 branch July 24, 2024 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pex external venv should copy wheel files
2 participants