Skip to content

Use solution filter instead of multiple solutions#47531

Draft
sharwell wants to merge 2 commits intodotnet:mainfrom
sharwell:use-slnf
Draft

Use solution filter instead of multiple solutions#47531
sharwell wants to merge 2 commits intodotnet:mainfrom
sharwell:use-slnf

Conversation

@sharwell
Copy link
Copy Markdown
Contributor

@sharwell sharwell commented Sep 8, 2020

  • Change Compilers.sln references to Compilers.slnf
  • Update Compilers.slnf to include all items previously present in Compilers.sln
  • Remove Compilers.sln

The following items were present in Compilers.slnf even though they were not present in Compilers.sln. I did not remove these items:

@sharwell sharwell requested a review from a team as a code owner September 8, 2020 19:21
@sharwell sharwell marked this pull request as draft September 8, 2020 19:21
@genlu
Copy link
Copy Markdown
Member

genlu commented Sep 8, 2020

slnf doesn't seem to work with the version of dotnet build we use for CI

@jaredpar
Copy link
Copy Markdown
Member

jaredpar commented Sep 8, 2020

That should get fixed when we move to a new .NET SDK version, believe @333fred already has a PR out to do just that. But we're blocked on a number of issues to get that in. 😦

Copy link
Copy Markdown
Member

@333fred 333fred left a comment

Choose a reason for hiding this comment

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

Solution filters do not work with vscode.

@sharwell
Copy link
Copy Markdown
Contributor Author

sharwell commented Sep 9, 2020

Solution filters do not work with vscode.

Several other repositories in the organization have already switched to using solution filters. Can you be more specific?

@333fred
Copy link
Copy Markdown
Member

333fred commented Sep 9, 2020

Several other repositories in the organization have already switched to using solution filters. Can you be more specific?

If they have, then they do not have users that work in vscode that regularly use the solutions they removed. It's certainly possible that we could add the feature to omnisharp, but today omnisharp does not support loading slnf files.

"src\\Compilers\\VisualBasic\\Test\\Symbol\\Microsoft.CodeAnalysis.VisualBasic.Symbol.UnitTests.vbproj",
"src\\Compilers\\VisualBasic\\Test\\Syntax\\Microsoft.CodeAnalysis.VisualBasic.Syntax.UnitTests.vbproj",
"src\\Compilers\\VisualBasic\\vbc\\vbc.csproj",
"src\\Dependencies\\CodeAnalysis.Debugging\\Microsoft.CodeAnalysis.Debugging.shproj",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is another issue that affects Mac and Linux users that we need to take into account - dotnet/msbuild#5715

~/Source/roslyn [master ≡ +1 ~0 -0 !]> dotnet restore "./Compilers.slnf"
/Users/joeyrobichaud/Source/roslyn/src/Analyzers/CSharp/Analyzers/CSharpAnalyzers.shproj : Solution file error MSB5028: Solution filter file at "System.Collections.Generic.HashSet`1[System.String]" includes project "src\Analyzers\CSharp\Analyzers\CSharpAnalyzers.shproj" that is not in the solution file at "/Users/joeyrobichaud/Source/roslyn/Roslyn.sln".

@Youssef1313
Copy link
Copy Markdown
Member

@sharwell @333fred VSCode now supports slnf (OmniSharp/omnisharp-roslyn#2121). I think it should be possible to get this in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants