Skip to content

gh-131591: Reset RemoteDebuggerSuupport state after fork#132793

Merged
pablogsal merged 4 commits intopython:mainfrom
Zheaoli:manjusaka/reset-fork-remote
Apr 25, 2025
Merged

gh-131591: Reset RemoteDebuggerSuupport state after fork#132793
pablogsal merged 4 commits intopython:mainfrom
Zheaoli:manjusaka/reset-fork-remote

Conversation

@Zheaoli
Copy link
Contributor

@Zheaoli Zheaoli commented Apr 22, 2025

FYI I think we need to reset process _PyRemoteDebuggerSupport state after os.fork. Otherwise, the child process may inherit the pendding_call == 1 state

Signed-off-by: Manjusaka <me@manjusaka.me>
@Zheaoli
Copy link
Contributor Author

Zheaoli commented Apr 24, 2025

@pablogsal PTAL when you got time

@pablogsal
Copy link
Member

@godlygeek WDYT?

Zheaoli and others added 3 commits April 24, 2025 20:27
…e-131591.CdEqBr.rst

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
Signed-off-by: Manjusaka <me@manjusaka.me>
@Zheaoli Zheaoli force-pushed the manjusaka/reset-fork-remote branch from 8288a7e to 16f3f24 Compare April 24, 2025 12:31
@Zheaoli Zheaoli requested a review from pablogsal April 24, 2025 15:27
@godlygeek
Copy link
Contributor

FYI I think we need to reset process _PyRemoteDebuggerSupport state after os.fork.

I agree, good catch!

@pablogsal pablogsal merged commit 3a39e33 into python:main Apr 25, 2025
46 checks passed
@pablogsal
Copy link
Member

Thanks a lot for your contribution @Zheaoli !

@Zheaoli Zheaoli deleted the manjusaka/reset-fork-remote branch April 25, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants