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-29191: Add liblzma.vcxproj to pcbuild.sln and other missing entries #1222

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

segevfiner
Copy link
Contributor

liblzma is missing from pcbuild.sln. This causes the build of _lzma to fail when building the solution and not using build.bat.

A similar patch is simple enough to create for older versions/branches, in case this doesn't back port cleanly. Just Add->Existing Project liblzma.vcxproj and make sure to revert any lines in the sln which Visual Studio added that you don't want to commit (Like VisualStudioVersion = ...).

liblzma is missing from pcbuild.sln. This causes the build of _lzma to
fail when building the solution and not using build.bat.
@mention-bot
Copy link

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

@zooba zooba merged commit f60c9e5 into python:master Apr 20, 2017
@segevfiner segevfiner deleted the issue-29191-sln-fix branch April 20, 2017 23:39
zooba pushed a commit to zooba/cpython that referenced this pull request Apr 21, 2017
…es (python#1222)

liblzma is missing from pcbuild.sln. This causes the build of _lzma to fail when building the solution and not using build.bat.
Mariatta pushed a commit that referenced this pull request Apr 21, 2017
…es (GH-1222) (GH-1244)

liblzma is missing from pcbuild.sln. This causes the build of _lzma to fail when building the solution and not using build.bat.
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.

4 participants