Skip to content
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.

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

Alternarive for bpo-29553 - Fixed ArgumentParses format_usage for mutually exclusive groups #120

Closed
wants to merge 4 commits into from
Closed

Alternarive for bpo-29553 - Fixed ArgumentParses format_usage for mutually exclusive groups #120

wants to merge 4 commits into from

Conversation

andrewnester
Copy link
Contributor

Fix for https://bugs.python.org/issue29553
Alternative to #117
This alternative advised by Christoph Stahl and Brian Curtin

@andrewnester andrewnester changed the title bpo-29553 - Fixed ArgumentParses format_usage for mutually exclusive groups Alternarive for bpo-29553 - Fixed ArgumentParses format_usage for mutually exclusive groups Feb 15, 2017
Copy link
Member

@berkerpeksag berkerpeksag left a comment

Choose a reason for hiding this comment

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

See #117 (comment) about tests I left in #117.

@andrewnester
Copy link
Contributor Author

@berkerpeksag thanks! just updated my PR.

Misc/NEWS Outdated
@@ -251,6 +251,7 @@ Library

- bpo-29576: Improve some deprecations in importlib. Some deprecated methods
now emit DeprecationWarnings and have better descriptive messages.
Copy link
Member

Choose a reason for hiding this comment

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

We need an empty line here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zhangyangyu updated, sorry! thanks!

akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
Stackless now correctly grabs and releases the head_mutex, if it
iterates over the linked list of Py_ThreadState structures.

https://bitbucket.org/stackless-dev/stackless/issues/120
(grafted from 1fb75d58a98083b2547746adb21fa238bbf8135a)
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
This commit fixes a regression introduced by commit fc13eecdd0ad.
When I grafted the patch for this issue from branch 2.7-slp (1fb75d58a), I forgot to remove a 2.7-slp only part of the code.
In effect, this change reapplies commit b8a8122bbb91.

https://bitbucket.org/stackless-dev/stackless/issues/120
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
@brettcannon
Copy link
Member

To try and help move older pull requests forward, we are going through and backfilling 'awaiting' labels on pull requests that are lacking the label. Based on the current reviews, the best we can tell in an automated fashion is that a core developer requested changes to be made to this pull request.

If/when the requested changes have been made, please leave a comment that says, I have made the requested changes; please review again. That will trigger a bot to flag this pull request as ready for a follow-up review.

@berkerpeksag
Copy link
Member

Superseded by PR #14976.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants