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
Upgrade to pex 1.6.8. #7962
Upgrade to pex 1.6.8. #7962
Conversation
""" | ||
chosen_interpreter_binary_path = interpreter.binary | ||
return { | ||
'PEX_IGNORE_RCFILES': '1', | ||
'PEX_PYTHON': chosen_interpreter_binary_path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NB: The relevant logic is here: https://github.com/pantsbuild/pex/blob/f5e00ad07a4943b101b4d17f94a1146856cfb585/pex/pex_bootstrapper.py#L131-L173
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay simplicity!
This picks up fixes to `PEX_PYTHON` and `PEX_PYTHON_PATH` handling and allows us to simplify pex execution slightly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have started an internal testing battery against this change, to see if we're missing any case with test coverage. Assuming that's well and good, lgtm.
This is a follow-up to pantsbuild#7962 [ci skip] # Documentation-only change.
This is a follow-up to #7962 [ci skip] # Documentation-only change.
This picks up fixes to
PEX_PYTHON
andPEX_PYTHON_PATH
handling andallows us to simplify pex execution slightly.