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.

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

Merged
merged 16 commits into from
Mar 7, 2025

Conversation

surayya-MS
Copy link
Member

@surayya-MS surayya-MS commented Mar 4, 2025

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

  1. Reverted Revert ".slnx support - use the new parser for .sln and .slnx (#10836)"
  2. Added 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.
  3. Fixed tests

Testing

Changed tests to opt-in to .sln parsing with the new parser from Microsoft.VisualStudio.SolutionPersistence.

Notes

@surayya-MS surayya-MS requested a review from a team as a code owner March 4, 2025 18:19
@surayya-MS
Copy link
Member Author

the changes are exactly same as in https://github.com/dotnet/msbuild/pull/11487/files#diff-047e3c8fd3315f9a08db8ca4fabd30b15f324080bebc140fe1652e5d8f8af4d1
except for the version bump. You can chek this for easier review

@surayya-MS surayya-MS self-assigned this Mar 4, 2025
@akoeplinger
Copy link
Member

Can we also update the SolutionPersistence package to the latest version https://github.com/microsoft/vs-solutionpersistence/releases/tag/v1.0.52 ?

<MicrosoftVisualStudioSolutionPersistenceVersion>1.0.28</MicrosoftVisualStudioSolutionPersistenceVersion>

It has a fix for an issue that broke a test in the sdk repo.

@surayya-MS
Copy link
Member Author

Can we also update the SolutionPersistence package to the latest version https://github.com/microsoft/vs-solutionpersistence/releases/tag/v1.0.52 ?

<MicrosoftVisualStudioSolutionPersistenceVersion>1.0.28</MicrosoftVisualStudioSolutionPersistenceVersion>

It has a fix for an issue that broke a test in the sdk repo.

Yes, we'll update in the next PR

@kasperk81
Copy link
Contributor

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)

@surayya-MS surayya-MS merged commit 0e528f7 into dotnet:main Mar 7, 2025
10 checks passed
@maridematte
Copy link
Member

/backport to vs17.14

Copy link
Contributor

Started backporting to vs17.14: https://github.com/dotnet/msbuild/actions/runs/13788533134

Copy link
Contributor

@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

@maridematte
Copy link
Member

/backport to vs17.14

Copy link
Contributor

Started backporting to vs17.14: https://github.com/dotnet/msbuild/actions/runs/13788605583

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.

6 participants