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

Use solution filter instead of multiple solutions #47531

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sharwell
Copy link
Member

@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
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
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
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
Member 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
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.

@@ -45,20 +45,35 @@
"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
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
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