- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.4k
Opt-in .sln parsing with Microsoft.VisualStudio.SolutionPersistence #11538
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
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) | 
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_OPTINas 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