-
Notifications
You must be signed in to change notification settings - Fork 182
Fixes Autolinking on new boost version #510
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
base: develop
Are you sure you want to change the base?
Conversation
|
I think that is not the correct fix because MSVC 19.40 is still |
I don't think so, We had a bug where this fixed it, but it is now moot to me, so happy for this not to go in. I am on windows so happy to download the latest pre-release and find out if you'd like 😄 |
|
I'm looking into this now, we need to do exactly the same thing as Boost.Build. |
|
So far as I can tell from current develop, I still get "143" in library names, I haven't been able to spot where Boost.Build sets this though. Attempting to build with toolset=msvc-14.4 produces: Can you confirm? |
That's weird because if i set Not sure if it is related but i am also calling |
This is what happened to us. I have experienced oddities with setting |
I don't see how you can do that, I checked the source (https://github.com/boostorg/build/blob/develop/src/tools/msvc.jam) and there is no such thing as a "14.4" version, and for me attempting to use it, generates a hard error from b2 (which it should). This looks like user error to me since we only support 14.3 as a valid version not 14.4. I don't know how you managed to slip by the error checks though. @grafikrobot would you have any idea what's happening at the b2 end of things? |
Looks like https://github.com/boostorg/build/blob/develop/src/tools/msvc.jam#L1157 does in fact allow a value of
Which toolsets do you have installed on your system (I've got I think the main issue is, that Microsoft broke the assumption that you can derive the 'Platform Toolset' version (e.g. See https://devblogs.microsoft.com/cppblog/msvc-toolset-minor-version-number-14-40-in-vs-2022-v17-10/ for more information. This even caused some issues in their build systems as well. In our case, we'd like to build with the B2 is probably deriving the name just from the toolset version, which is now broken. |
|
So I just updated to the latest and greatest VS-17.14, which has msvc version 194435217. For me Boost.Build is still identifying that as msvc-14.3 and naming binaries appropriately. However.... the update has mangled something badly, as the IDE will no longer build C++ programs (that's the second update in a row that's done that, I'll have to completely remove and reinstall to fix - when I have the time). Meanwhile I'd still like to hear from @grafikrobot what's intended in Boost.Build, as ultimately the only thing that matters is that we do the same thing. |
|
@pdimov might have a view also. |
|
Even though the toolset version is 19.4x, it's still toolset v143, which corresponds to msvc-14.3. There's no such thing as msvc-14.4. The new Visual Studio will be v145, msvc-14.5. |
Fixes #496