X Tutup
The Wayback Machine - https://web.archive.org/web/20221223113711/https://github.com/python/cpython/pull/27857
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

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

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

the-knights-who-say-ni commented Aug 20, 2021

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

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 Sep 8, 2021
@florinspatar
Copy link
Contributor Author

florinspatar commented Sep 8, 2021

Did the requested NEWS change. test_multiprocessing_fork seem to fail randomly

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

vstinner commented Sep 8, 2021

Merged, thanks for your contribution @florinspatar!

@miss-islington
Copy link
Contributor

miss-islington commented Oct 22, 2021

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

miss-islington commented Oct 22, 2021

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

@bedevere-bot
Copy link

bedevere-bot commented Oct 22, 2021

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

@bedevere-bot
Copy link

bedevere-bot commented Oct 22, 2021

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
…UX (pythonGH-27857)

(cherry picked from commit 2396fa6)

Co-authored-by: Florin Spătar <florin.spatar@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2021
…UX (pythonGH-27857)

(cherry picked from commit 2396fa6)

Co-authored-by: Florin Spătar <florin.spatar@gmail.com>
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 <florin.spatar@gmail.com>
ambv pushed a commit that referenced this pull request Nov 18, 2021
…UX (GH-27857) (GH-29152)

(cherry picked from commit 2396fa6)

Co-authored-by: Florin Spătar <florin.spatar@gmail.com>
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.

None yet

5 participants
X Tutup