Skip to content

Conversation

@orenmn
Copy link
Contributor

@orenmn orenmn commented Sep 14, 2017

  • in _winapi.c: move the check whether PyMapping_Keys() failed to right after calling it.
  • in test_subprocess.py: add a test to verify that the assertion failure is no more.

https://bugs.python.org/issue31471

@serhiy-storchaka serhiy-storchaka added needs backport to 3.6 type-bug An unexpected behavior, bug, or error labels Sep 14, 2017
@serhiy-storchaka serhiy-storchaka merged commit 0b3a87e into python:master Sep 14, 2017
@miss-islington
Copy link
Contributor

Thanks @orenmn for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 14, 2017
…ws, in case env has a bad keys() method. (pythonGH-3580)

(cherry picked from commit 0b3a87e)
@bedevere-bot
Copy link

GH-3584 is a backport of this pull request to the 3.6 branch.

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

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants