Skip to content

gh-141004: Document Py_FrozenMain - #159724

Merged
encukou merged 4 commits into
python:mainfrom
ZeroIntensity:docs/c-api/document-frozenmain
Oct 20, 2026
Merged

encukou merged 4 commits into
python:mainfrom
ZeroIntensity:docs/c-api/document-frozenmain

Conversation

@ZeroIntensity

@ZeroIntensity ZeroIntensity commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

@read-the-docs-community

read-the-docs-community Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

@ZeroIntensity
ZeroIntensity requested a review from encukou October 19, 2026 18:42
Comment thread Doc/c-api/interp-lifecycle.rst Outdated
Return ``0`` if ``__main__`` ran without an unhandled exception, ``1``
otherwise, and ``120`` if finalizing the interpreter failed (see
:c:func:`Py_FinalizeEx`).
An unhandled :exc:`SystemExit` terminates the process.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this right? (I'll need to check or study the code a bit more to verify; but if you already did, I'll believe you :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Admittedly, Codex did a lot of the research on the details here. I'll double-check to make sure that it's not hallucinating.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think this line is correct.

@github-project-automation github-project-automation Bot moved this from Todo to In Progress in Sprint Oct 20, 2026
@encukou
encukou enabled auto-merge (squash) October 20, 2026 17:40
@encukou

encukou commented Oct 20, 2026

Copy link
Copy Markdown
Member

Thank you!

@encukou
encukou merged commit 6f439ee into python:main Oct 20, 2026
43 checks passed
@miss-islington-app

Copy link
Copy Markdown

Thanks @ZeroIntensity for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖

@github-project-automation github-project-automation Bot moved this from In Progress to Done in Sprint Oct 20, 2026
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Oct 20, 2026
@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159922 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 20, 2026
@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159923 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 20, 2026
@ZeroIntensity
ZeroIntensity deleted the docs/c-api/document-frozenmain branch October 20, 2026 17:45
ZeroIntensity added a commit that referenced this pull request Oct 20, 2026
(cherry picked from commit 6f439ee)

Co-authored-by: Peter Bierma <[email protected]>
ZeroIntensity added a commit that referenced this pull request Oct 20, 2026
(cherry picked from commit 6f439ee)

Co-authored-by: Peter Bierma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news sprint

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants