-
Notifications
You must be signed in to change notification settings - Fork 1.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
Opt-in .sln parsing with Microsoft.VisualStudio.SolutionPersistence #11538
Conversation
…ce in ChangeWaves
the changes are exactly same as in https://github.com/dotnet/msbuild/pull/11487/files#diff-047e3c8fd3315f9a08db8ca4fabd30b15f324080bebc140fe1652e5d8f8af4d1 |
Can we also update the SolutionPersistence package to the latest version https://github.com/microsoft/vs-solutionpersistence/releases/tag/v1.0.52 ? Line 43 in d047377
It has a fix for an issue that broke a test in the sdk repo. |
Yes, we'll update in the next PR |
src/Build.UnitTests/Construction/SolutionFile_OldParser_Tests.cs
Outdated
Show resolved
Hide resolved
dotnet/sdk#47328 is upgrading to 1.0.52 for 9.0.2 which has fixes for the issues reported in msbuild issue tracker (source-build-externals is also updated) |
/backport to vs17.14 |
Started backporting to vs17.14: https://github.com/dotnet/msbuild/actions/runs/13788533134 |
@maridematte an error occurred while backporting to "vs17.14", please check the run log for details! Error: @maridematte is not a repo collaborator, backporting is not allowed. If you're a collaborator please make sure your dotnet team membership visibility is set to Public on https://github.com/orgs/dotnet/people?query=maridematte |
/backport to vs17.14 |
Started backporting to vs17.14: https://github.com/dotnet/msbuild/actions/runs/13788605583 |
Context
Require opt-in (instead of -out) to .sln parsing with the new parser from Microsoft.VisualStudio.SolutionPersistence.
this is #11487 targeted 17.14. We decided to revert the change in 17.13
Changes Made
MSBUILD_SLN_PARSING_SOLUTIONPERSISTENCE_OPTIN
as an opt-in environment variable instead of requiring changewave opt-out to return to the MSBuild-internal solution parser.Testing
Changed tests to opt-in to .sln parsing with the new parser from Microsoft.VisualStudio.SolutionPersistence.
Notes