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

Handle dlopen(NULL) failure in glibc fallback #13

Merged
merged 2 commits into from
Jul 23, 2024

Conversation

jsirois
Copy link
Member

@jsirois jsirois commented Jul 23, 2024

(cherry picked from commit 601bcf8)

Manually removed type annotation added in that commit to keep Python 2.7
support for Pex vendored Pip.

Co-authored-by: Pradyun Gedam [email protected]

charliermarsh and others added 2 commits July 23, 2024 16:07
This is needed to maintain Python 2.7 support.
@jsirois jsirois requested a review from benjyw July 23, 2024 23:14
@jsirois
Copy link
Member Author

jsirois commented Jul 23, 2024

@benjyw I'm going to submit this TBR, which is safe, since Pex pins to the prior tip sha of this patches branch. There will be a separate PR coming from Pex to bump to this sha to supprt musl libc static Pythons, like those PBS ships.

@jsirois jsirois merged commit 00827ec into pex/patches/generation-2 Jul 23, 2024
1 of 7 checks passed
@jsirois jsirois deleted the musl/fix branch July 23, 2024 23:17
jsirois added a commit to pex-tool/pex that referenced this pull request Jul 24, 2024
This pulls in a new vendored version of Pip with
the recent fix in pypa/pip#12716 applied in
pex-tool/pip#13. As a result, Pex can run using
vendored Pip under statically linked musl libc CPython interpreters.
This opens the door to bootstrapping newer unpatched Pip's that also
have this same fix (versions 24.2 and later; see: #2471).

Fixes #2017
@benjyw
Copy link

benjyw commented Jul 24, 2024

@benjyw I'm going to submit this TBR, which is safe, since Pex pins to the prior tip sha of this patches branch. There will be a separate PR coming from Pex to bump to this sha to supprt musl libc static Pythons, like those PBS ships.

SG. I'm back from Europe now, so will be more available for reviews, and at normal hours.

Copy link

@benjyw benjyw left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants