Skip to content

[main] Source code updates from dotnet/dotnet#536

Open
dotnet-maestro[bot] wants to merge 7 commits intomainfrom
darc-main-17a449b0-f1de-4c8c-ad80-9e6739ba482f
Open

[main] Source code updates from dotnet/dotnet#536
dotnet-maestro[bot] wants to merge 7 commits intomainfrom
darc-main-17a449b0-f1de-4c8c-ad80-9e6739ba482f

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Mar 18, 2026

Note

This is a codeflow update. It may contain both source code changes from
the VMR
as well as dependency updates. Learn more here.

This pull request brings the following source code changes

From https://github.com/dotnet/dotnet

Updated Dependencies

Associated changes in source repos

Diff the source with this PR branch
darc vmr diff --name-only https://github.com/dotnet/dotnet:803eb28628f5623c108f1bf33504c86e19815821..https://github.com/dotnet/deployment-tools:darc-main-17a449b0-f1de-4c8c-ad80-9e6739ba482f

Updated Dependencies:
Microsoft.DotNet.Arcade.Sdk (Version 11.0.0-beta.26070.110 -> 11.0.0-beta.26167.112)
[[ commit created by automation ]]
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner March 18, 2026 08:33
@akoeplinger akoeplinger enabled auto-merge (squash) March 18, 2026 08:34
libnethost.lib in runtime is built by VS2026/v145 toolset so linking fails when we don't bump.
Comment on lines -72 to -75
:VS2022
:: Setup vars for VS2022
set __PlatformToolset=v143
set __VSVersion=17 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@akoeplinger would we not need this for VS 2022 case?

Copy link
Member

@akoeplinger akoeplinger Mar 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NikolaMilosavljevic yes, it's there on line 63

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The concern is with __PlatformToolset that is now always set to v145

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NikolaMilosavljevic I think that's required because libnethost.lib (which is built in dotnet/runtime using VS2026) would fail to link otherwise, that was the original issue in the PR:

libnethost.lib(utils.cpp.obj) : error LNK2019: unresolved external symbol __std_find_first_not_of_trivial_pos_2 referenced in function "unsigned __int64 __cdecl std::_Find_first_not_of_pos_vectorized<wchar_t,wchar_t>(wchar_t const * const,unsigned __int64,wchar_t const * const,unsigned __int64)" (??$_Find_first_not_of_pos_vectorized@_W_W@std@@YA_KQEB_W_K01@Z) [D:\a_work\1\s\artifacts\obj\win-x64.Debug\native\projects\NetCoreCheck\EXE\NetCoreCheck.vcxproj]

Updated Dependencies:
Microsoft.DotNet.Arcade.Sdk (Version 11.0.0-beta.26167.112 -> 11.0.0-beta.26168.106)
[[ commit created by automation ]]
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.

2 participants