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

bpo-30726: Also fix pyexpat.vcxproj #2375

Merged
merged 1 commit into from
Jun 28, 2017

Conversation

segevfiner
Copy link
Contributor

@segevfiner segevfiner commented Jun 24, 2017

@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

@segevfiner, thanks for your PR! By analyzing the history of the files in this pull request, we identified @briancurtin and @zooba to be potential reviewers.

@segevfiner
Copy link
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

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

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
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
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
Copy link
Member

vstinner commented Sep 7, 2017

Backport to 2.7 already done by commit ab3b0ad. I removed the label.

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