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.

Sign up for GitHub

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

Jump to bottom

WinBuild : Documentation does not include VC 16 #7253

Closed
HughMacdonald opened this issue Jun 14, 2021 · 2 comments
Closed

WinBuild : Documentation does not include VC 16 #7253

HughMacdonald opened this issue Jun 14, 2021 · 2 comments
Labels
documentation Windows Windows-specific

Comments

@HughMacdonald
Copy link

This is a tiny thing, and apologies if I've misunderstood anything.

I'm running Visual Studio 16.10.0 here, which I'm guessing is the same version as VC. In the winbuild/README.md, the options for the -VC flag include up to 15, but should probably include 16 too. Running it with -VC=16 worked fine for me.
@bagder bagder added documentation Windows Windows-specific labels Jun 14, 2021
@bagder
Copy link
Member

bagder commented Jun 14, 2021 •
edited

It probably should, yes. Or be rephrased to work generically since the only check that's currently done with that variable is if it is version 6, all others are treated the same...

bagder added a commit that referenced this issue Jun 14, 2021
@bagder
winbuld/README: VC should be set to 6 'or larger'
34c0906 
Previously it listed all versions up to 15 (missing 16) but this new
phrasing is more open ended.

Fixes #7253
Reported-by: Hugh Macdonald
@bagder bagder mentioned this issue Jun 14, 2021
Closed
@HughMacdonald
Copy link
Author

Cool stuff - thanks! It only threw me for a second, and it seemed to build just fine.

@bagder bagder closed this as completed in ce8a38f Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Windows Windows-specific
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

winbuild/README: VC should be set to 6 'or larger' curl/curl
2 participants
@bagder @HughMacdonald