bpo-30726: Also fix pyexpat.vcxproj by segevfiner · Pull Request #2375 · python/cpython · GitHub
Skip to content

bpo-30726: Also fix pyexpat.vcxproj - #2375

Merged
zooba merged 1 commit into
python:masterfrom
segevfiner:bpo-30726-pyexpat-vcxproj
Jun 28, 2017
Merged

zooba merged 1 commit into
python:masterfrom
segevfiner:bpo-30726-pyexpat-vcxproj

Conversation

@segevfiner

@segevfiner segevfiner commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

@Haypo

We have the same issue that you fixed in _elementtree.vcxproj in pyexpat.vcxproj. It doesn't seem like anyone in the CPython core links to this one. But it does produce a static library that is bundled with the CPython distribution. 😖

@mention-bot

Copy link
Copy Markdown

@segevfiner

Copy link
Copy Markdown
Contributor Author

ping @Haypo

@zooba
zooba merged commit 7526cad into python:master Jun 28, 2017
segevfiner added a commit to segevfiner/cpython that referenced this pull request Jul 4, 2017
@bedevere-bot

Copy link
Copy Markdown

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

segevfiner added a commit to segevfiner/cpython that referenced this pull request Jul 4, 2017
@bedevere-bot

Copy link
Copy Markdown

GH-2571 is a backport of this pull request to the 3.5 branch.

vstinner pushed a commit that referenced this pull request Jul 4, 2017
vstinner pushed a commit that referenced this pull request Jul 4, 2017
@segevfiner
segevfiner deleted the bpo-30726-pyexpat-vcxproj branch July 4, 2017 19:26
ned-deily pushed a commit to ned-deily/cpython that referenced this pull request Jul 7, 2017
(cherry picked from commit 7526cad)
(cherry picked from commit d02c841)
@miss-islington

Copy link
Copy Markdown
Contributor

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

@miss-islington

Copy link
Copy Markdown
Contributor

Sorry, @segevfiner and @zooba, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.

@vstinner

vstinner commented Sep 7, 2017

Copy link
Copy Markdown
Member

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.

8 participants