bpo-36250: ignore ValueError from signal in non-main thread by blueyed · Pull Request #12251 · python/cpython · GitHub
Skip to content

bpo-36250: ignore ValueError from signal in non-main thread - #12251

Merged
zware merged 2 commits into
python:masterfrom
blueyed:pdb-interaction-signal
Sep 9, 2019
Merged

zware merged 2 commits into
python:masterfrom
blueyed:pdb-interaction-signal

Conversation

@blueyed

@blueyed blueyed commented Mar 9, 2019

Copy link
Copy Markdown
Contributor

Comment thread Lib/pdb.py Outdated
Comment thread Lib/pdb.py Outdated
@bedevere-bot

Copy link
Copy Markdown

Co-Authored-By: blueyed <github@thequod.de>
@blueyed

blueyed commented Mar 10, 2019

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again.

@bedevere-bot

Copy link
Copy Markdown

Thanks for making the requested changes!

@zware: please review the changes made to this pull request.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @blueyed for the PR, and @zware for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2019
…-12251)

Authored-By: blueyed <github@thequod.de>
(cherry picked from commit 8d64bfa)

Co-authored-by: Daniel Hahler <github@thequod.de>
@bedevere-bot

Copy link
Copy Markdown

GH-15752 is a backport of this pull request to the 3.8 branch.

@zware

zware commented Sep 9, 2019

Copy link
Copy Markdown
Member

miss-islington added a commit that referenced this pull request Sep 9, 2019
Authored-By: blueyed <github@thequod.de>
(cherry picked from commit 8d64bfa)

Co-authored-by: Daniel Hahler <github@thequod.de>
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
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.

5 participants