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

Jump to bottom

bpo-45847: Port _multiprocessing to PY_STDLIB_MOD (GH-29768) #29768

Merged
merged 2 commits into from Nov 28, 2021
Merged

bpo-45847: Port _multiprocessing to PY_STDLIB_MOD (GH-29768) #29768

merged 2 commits into from Nov 28, 2021

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Nov 25, 2021 •
edited by bedevere-bot

tiran
tiran reviewed Nov 25, 2021
View changes
configure.ac Outdated Show resolved Hide resolved
@erlend-aasland erlend-aasland force-pushed the ac-multiproc branch 2 times, most recently from 36cf95f to 91c7cd9 Compare Nov 25, 2021
@erlend-aasland erlend-aasland marked this pull request as ready for review Nov 25, 2021
@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented Nov 25, 2021 •
edited

AFAICS, HAVE_SEM_OPEN and POSIX_SEMAPHORES_NOT_ENABLED are handled by #ifdefs in the source code. We could add a check for $ac_cv_header_fcntl_h, but it's probably not needed.

@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 26, 2021
@bedevere-bot
Copy link

bedevere-bot commented Nov 26, 2021

🤖 New build scheduled with the buildbot fleet by @tiran for commit 0af5f50 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 26, 2021
tiran
tiran approved these changes Nov 26, 2021
View changes
Copy link
Member

@tiran tiran left a comment

Thanks, looks good to me.

I'm unsure about the semaphore extension. It is hard to test.

@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented Nov 26, 2021

I'm unsure about the semaphore extension.

Me too. Let's ask on python-dev / Discourse.

@erlend-aasland erlend-aasland marked this pull request as draft Nov 27, 2021
@erlend-aasland erlend-aasland marked this pull request as ready for review Nov 27, 2021
tiran
tiran approved these changes Nov 27, 2021
View changes
@tiran tiran changed the title bpo-45847: Port _multiprocessing to PY_STDLIB_MOD bpo-45847: Port _multiprocessing to PY_STDLIB_MOD (GH-29768) Nov 28, 2021
@tiran tiran merged commit aaf4222 into python:main Nov 28, 2021
11 checks passed
@erlend-aasland erlend-aasland deleted the ac-multiproc branch Nov 28, 2021
@tiran tiran mentioned this pull request Apr 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tiran tiran

Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@erlend-aasland @bedevere-bot @tiran @the-knights-who-say-ni