Skip to content

Add support for multi-phase module initialization#485

Open
o01eg wants to merge 5 commits intoboostorg:developfrom
o01eg:multi-phase-module-init
Open

Add support for multi-phase module initialization#485
o01eg wants to merge 5 commits intoboostorg:developfrom
o01eg:multi-phase-module-init

Conversation

@o01eg
Copy link
Copy Markdown

@o01eg o01eg commented Apr 20, 2025

Fixes #484

@o01eg
Copy link
Copy Markdown
Author

o01eg commented May 11, 2025

I only unsure if Py_mod_create should be accessible and defined too.

@o01eg
Copy link
Copy Markdown
Author

o01eg commented Nov 14, 2025

Ping!

@o01eg o01eg marked this pull request as draft April 4, 2026 14:25
@o01eg
Copy link
Copy Markdown
Author

o01eg commented Apr 4, 2026

It's quite bad that nogil support was added only to single-phase initialization

@o01eg o01eg force-pushed the multi-phase-module-init branch from 957157e to 40105f0 Compare April 4, 2026 18:59
@o01eg o01eg marked this pull request as ready for review April 4, 2026 19:00
@o01eg
Copy link
Copy Markdown
Author

o01eg commented Apr 4, 2026

Ping! I need an approval.

@o01eg
Copy link
Copy Markdown
Author

o01eg commented Apr 5, 2026

Looks like Windows CI is broken not by my changes:

  A suitable version of 7zip was not found (required v21.7.0). Downloading portable 7zip v21.7.0...
  Downloading 7zip...
    https://www.7-zip.org/a/7z2107-extra.7z -> D:\a\python\vcpkg\downloads\7z2107-extra.7z
  Error: Failed to download from mirror set:
  https://www.7-zip.org/a/7z2107-extra.7z: failed: status code 404

What I see on https://www.7-zip.org/download.html there no such version.

@o01eg o01eg force-pushed the multi-phase-module-init branch from b43199f to 542974f Compare April 5, 2026 13:13
@o01eg
Copy link
Copy Markdown
Author

o01eg commented Apr 5, 2026

I've removed unsuccessful try to fix Windows CI. At least other OS passed.

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.

Support for multi-phase module initialization with PyModule_FromDefAndSpec

1 participant