-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
build-openssl.bat: Refer to VS2017 as VC14.1 instead of VC15 #2189
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it's VS 15, but VC 14.1 or 14.10/11/12. And CL 19.10/11/12. But they promised to get rid of that version mess for the next major version :-)
Made obsolete with be03364? |
.. and do the same for build-wolfssl.bat. Because MS calls it VC14.1. Closes curl#2189
02413fc
to
26d0c9c
Compare
hm. Alright I rebased on master. The remaining issue is calling it VC14.1 instead of VC15. |
I believe 14.1 is the version of the compiler and also the toolchain. The VC in VC15 is kind of short for Visual C++ from the days before Visual Studio. Visual Studio 2017 itself is version 15. If memory serves me correctly it was known as Dev15 during development and VS "15" during beta - the latest non-preview release is 15.5.6 which I'm running. |
Yeah but what I'm saying is it's Visual C++ 14.1 not Visual C++ 15. This is the first time the VS number is different from the VC number (VS15 uses VC14.1) |
Lots of sources online say "14.1" for VS 2017. Anyone against? |
Splitting hairs:
So the compiler is version 19.12 The "Platform Toolset" is "Visual Studio 2017 (v141)" (no dots) but that is selectable (in the vcxproj file for msbuild systems, not sure for nmake) going all the way back to at least VC10 with and without XP support. The Platform is "Visual Studio 15" (mine is 15.5.6, 'cos that has the Spectre switches). I don't know where that leaves you, |
I'd agree with 14.1 because:
|
I’d +1 @MarcelRaad said. Fwiw |
In the PHP world they are retaining VC15, apparently because the CRT is VC15(0): Also, in the Apache for Windows world VC15 is the norm: So, I am -1 |
.. and do the same for build-wolfssl.bat.
Refer to VS2017 as VC14.1 instead of VC15.
Support Enterprise and Professional variants.
Closes #xxxx
Another WIP. @captain-caveman2k as far as I can tell VS2017 is referred to as VC14.1 not VC15. I had some related confusion about this last year and I asked on the MS forums whether it's 14.10 or 14.1 and they said 14.1.
Regarding Enterprise/Professional I haven't tested them yet but I assume they are the same. Basically I added an autodetect so the build script could find the version in order of preference Enterprise -> Professional -> Community
Also there is a Microsoft tool vswhere for more precise Visual Studio location for 2010+.