Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

brandtbucher
Copy link
Member

@brandtbucher brandtbucher commented May 19, 2023

PyOS_InputHook and PyOS_ReadlineFunctionPointer are now only called in the main interpreter.


📚 Documentation preview 📚: https://cpython-previews--104674.org.readthedocs.build/

@erlend-aasland
Copy link
Contributor

We should advertise this in What's New as well.

Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

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

LGTM

@brandtbucher brandtbucher enabled auto-merge (squash) May 22, 2023 19:25
@brandtbucher brandtbucher merged commit 357bed0 into python:main May 22, 2023
@arhadthedev
Copy link
Member

arhadthedev commented May 22, 2023

Should we backport it to 3.12?

Edit: I noticed that the label isn't added yet.

@JelleZijlstra
Copy link
Member

@brandtbucher this didn't actually end up in the 3.12 branch. Should we backport it, or is it a new feature that should only go into 3.13?

@brandtbucher brandtbucher added the needs backport to 3.12 only security fixes label May 22, 2023
@miss-islington
Copy link
Contributor

Thanks @brandtbucher for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 22, 2023
@bedevere-bot
Copy link

GH-104760 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label May 22, 2023
@brandtbucher
Copy link
Member Author

I was waiting for the label. 🙂

brandtbucher added a commit that referenced this pull request May 23, 2023
GH-104668: Don't call PyOS_* hooks in subinterpreters (GH-104674)
(cherry picked from commit 357bed0)

Co-authored-by: Brandt Bucher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs) topic-C-API topic-subinterpreters topic-tkinter type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants