Skip to content

[2.7] bpo-29656: Handle PR branches in 'make patchcheck'#628

Merged
ncoghlan merged 1 commit intopython:2.7from
ncoghlan:backport-482f7a2-2.7
Mar 12, 2017
Merged

[2.7] bpo-29656: Handle PR branches in 'make patchcheck'#628
ncoghlan merged 1 commit intopython:2.7from
ncoghlan:backport-482f7a2-2.7

Conversation

@ncoghlan
Copy link
Copy Markdown
Contributor

Additional changes needed to backport:

  • dropped legacy SVN support from patchcheck
  • use subprocess.PIPE to silence expected error output
  • don't try to use subprocess.Popen as a context manager
  • don't try to pass a keyword argument to str.split()

(cherry picked from commit 482f7a2)

Additional changes needed to backport:

- dropped legacy SVN support from patchcheck
- use subprocess.PIPE to silence expected error output
- don't try to use subprocess.Popen as a context manager
- don't try to pass a keyword argument to str.split()

(cherry picked from commit 482f7a2)
@ncoghlan
Copy link
Copy Markdown
Contributor Author

@brettcannon, @benjaminp I already merged the 3.x backports to make make patchcheck git PR-branch friendly, but there were a few more adjustments needed to get it to play nice with 2.7, so I'd like someone else to take a quick look before I land it.

@ncoghlan
Copy link
Copy Markdown
Contributor Author

On second thoughts, having a discrepancy in behaviour between 2.7 and the other active branches concerns me more than any potential glitches in the backport, so I've gone ahead and merged this one as well.

@ncoghlan ncoghlan deleted the backport-482f7a2-2.7 branch March 12, 2017 09:35
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.

2 participants