Skip to content

bpo-44959: Added fallback to extension modules with '.sl' suffix on HP-UX #27857

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

Merged
merged 4 commits into from
Sep 8, 2021

Conversation

florinspatar
Copy link
Contributor

@florinspatar florinspatar commented Aug 20, 2021

@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@florinspatar

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

@vstinner
Copy link
Member

vstinner commented Sep 7, 2021

I closed/reopened the PR to re-run the failed CI job: Address sanitizer (pull_request):

2 tests failed: test__xxsubinterpreters test_multiprocessing_fork

Please ping me once tests pass, I will merge your PR ;-)

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

The NEWS entry should be in the "Core" category, but I don't care much. Change it it you want (remove the old file, re-run blurb to add a new entry).

florinspatar and others added 2 commits September 8, 2021 11:27
@florinspatar
Copy link
Contributor Author

Did the requested NEWS change. test_multiprocessing_fork seem to fail randomly

@vstinner vstinner merged commit 2396fa6 into python:main Sep 8, 2021
@vstinner
Copy link
Member

vstinner commented Sep 8, 2021

Merged, thanks for your contribution @florinspatar!

@vstinner vstinner added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Oct 22, 2021
@miss-islington
Copy link
Contributor

Thanks @florinspatar for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @florinspatar for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Oct 22, 2021
@bedevere-bot
Copy link

GH-29152 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Oct 22, 2021
@bedevere-bot
Copy link

GH-29153 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2021
miss-islington added a commit that referenced this pull request Oct 22, 2021
…UX (GH-27857)

(cherry picked from commit 2396fa6)

Co-authored-by: Florin Spătar <[email protected]>
ambv pushed a commit that referenced this pull request Nov 18, 2021
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.

5 participants