Skip to content

bpo-36688: Adding an implementation of RLock in _dummy_thread #12943

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Jun 17, 2019
Merged

bpo-36688: Adding an implementation of RLock in _dummy_thread #12943

merged 3 commits into from
Jun 17, 2019

Conversation

vlabakje
Copy link
Contributor

@vlabakje vlabakje commented Apr 24, 2019 •
edited by bedevere-bot
Loading

Implementing an extremely simple RLock that extends Lock. It accepts multiple acquire() calls and just as many release() calls but no more. This should be sufficient for the _dummy_thread module and fixes issue 36688 where python -c 'import dummy_threading' would cause an ImportError.
I also added tests for RLock in Lib/test/test_dummy_thread and made it executable.

https://bugs.python.org/issue36688

@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for your contribution, we look forward to reviewing it!

auvipy
auvipy reviewed Apr 26, 2019
View reviewed changes
Copy link

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

probably a new entry needed?

skoslowski
skoslowski suggested changes May 6, 2019
View reviewed changes
Copy link
Contributor

@skoslowski skoslowski left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. I ran your tests and double-checked with the documentation

@csabella csabella requested a review from pitrou June 5, 2019 00:53
@vstinner vstinner merged commit c5905f3 into python:master Jun 17, 2019
@miss-islington
Copy link
Contributor

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

@miss-islington
Copy link
Contributor

Thanks @vlabakje for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 17, 2019
@vlabakje @miss-islington
bpo-36688: Adding an implementation of RLock in _dummy_thread (python…
d8a4808 
…GH-12943)

(cherry picked from commit c5905f3)

Co-authored-by: Joost Lek <vlabakje@gmail.com>
@bedevere-bot
Copy link

GH-14144 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 17, 2019
@vlabakje @miss-islington
bpo-36688: Adding an implementation of RLock in _dummy_thread (python…
5cf9248 
…GH-12943)

(cherry picked from commit c5905f3)

Co-authored-by: Joost Lek <vlabakje@gmail.com>
@bedevere-bot
Copy link

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

miss-islington added a commit that referenced this pull request Jun 17, 2019
@miss-islington @vlabakje
bpo-36688: Adding an implementation of RLock in _dummy_thread (GH-12943)
351b0e7 
(cherry picked from commit c5905f3)

Co-authored-by: Joost Lek <vlabakje@gmail.com>
miss-islington added a commit that referenced this pull request Jun 17, 2019
@miss-islington @vlabakje
bpo-36688: Adding an implementation of RLock in _dummy_thread (GH-12943)
ad50591 
(cherry picked from commit c5905f3)

Co-authored-by: Joost Lek <vlabakje@gmail.com>
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
@vlabakje @lisroach
bpo-36688: Adding an implementation of RLock in _dummy_thread (python…
5087bba 
…GH-12943)
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
@vlabakje @DinoV
bpo-36688: Adding an implementation of RLock in _dummy_thread (python…
5149433 
…GH-12943)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skoslowski skoslowski skoslowski requested changes

@auvipy auvipy auvipy left review comments

@pitrou pitrou Awaiting requested review from pitrou

Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@vlabakje @the-knights-who-say-ni @miss-islington @bedevere-bot @skoslowski @auvipy @vstinner