📦 Disable cythonize parallelism w/ spawn#770
Merged
webknjaz merged 2 commits intoansible:develfrom Oct 10, 2025
Merged
Conversation
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
f338c07 to
87e8744
Compare
webknjaz
added a commit
to webknjaz/ansible--pylibssh
that referenced
this pull request
Oct 1, 2025
psf-chronographer
bot
added
the
bot:chronographer:provided
This comment was marked as outdated.
This comment was marked as outdated.
webknjaz
added a commit
to webknjaz/ansible--pylibssh
that referenced
this pull request
Oct 2, 2025
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
87e8744 to
791ee57
Compare
webknjaz
added a commit
to webknjaz/ansible--pylibssh
that referenced
this pull request
Oct 2, 2025
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
791ee57 to
04f1fca
Compare
webknjaz
commented
Oct 2, 2025
webknjaz
added a commit
to webknjaz/ansible--pylibssh
that referenced
this pull request
Oct 10, 2025
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
04f1fca to
95686cf
Compare
webknjaz
added a commit
to webknjaz/ansible--pylibssh
that referenced
this pull request
Oct 10, 2025
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
95686cf to
cdb95b9
Compare
This is a workaround for a hang in Cython extention compilation under Python 3.8+ on macOS. It hangs in `multiprocessing` when the `spawn` method is set up so the patch simply sets the workers number to `1` to combat that. Fixes ansible#762 and fixes ansible#769. Upstream fix: cython/cython#7183
webknjaz
force-pushed
the
packaging/fix-build-hang-on-spawn-in-cythonize
branch
from
cdb95b9 to
c4af16a
Compare
github-project-automation
bot
moved this from 🫸In review🫷
to 🌈 Done 🦄
in 📅 Procrastinating in public 😵💫
webknjaz
commented
Jan 27, 2026
| # NOTE: script that hangs on macOS under Python 3.8+ due to the use of | ||
| # NOTE: the ``spawn`` method in ``multiprocessing``. This patch can be | ||
| # NOTE: reverted once the fix PR is usable upstream. | ||
| # Ref: https://github.com/cython/cython/pull/7183 |
Member
Author
There was a problem hiding this comment.
This is first released as a part of Cython 3.2.0b1. We should be able to patch this conditionally @ get_local_cythonize_config().
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a workaround for a hang in Cython extention compilation under Python 3.8+ on macOS. It hangs in
multiprocessingwhen thespawnmethod is set up so the patch simply sets the workers number to1to combat that.Fixes #762 and fixes #769.
Upstream fix: cython/cython#7183
ISSUE TYPE
Change log preview: https://ansible-pylibssh--770.org.readthedocs.build/en/770/changelog/#packaging-updates-and-notes-for-downstreams