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

Build MSBuild.SourceBuild.slnf (new solution filter) #1965

Closed
wants to merge 2 commits into from

Conversation

ladipro
Copy link
Member

@ladipro ladipro commented Jan 8, 2021

dotnet/msbuild#6010 replaced MSBuild.SourceBuild.sln with MSBuild.SourceBuild.slnf. The subset of projects it's building stays the same.

@crummel
Copy link
Contributor

crummel commented Jan 8, 2021

I updated so this PR is actually getting the version with the slnf file, but that might not be all it takes to fix it.

Copy link
Member

@dagood dagood left a comment

Choose a reason for hiding this comment

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

We don't have a branch for 6.0, so reacting to the slnf change isn't actually possible yet. Can let CI keep going to see if this brings up any forward-looking issues that need to be addressed though.

@@ -52,7 +52,7 @@
</Dependency>
<Dependency Name="Microsoft.Build" Version="16.8.0" CoherentParentDependency="Microsoft.Dotnet.Sdk.Internal">
<Uri>https://github.com/dotnet/msbuild</Uri>
<Sha>126527ff107ae93fed10af675506c56d046aa5a3</Sha>
<Sha>c6ed2fef9d1cb5504eb51c6f6a9d529e4efd9054</Sha>
Copy link
Member

Choose a reason for hiding this comment

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

Probably shouldn't update our 5.0 release branch (source-build master) to a 6.0 commit (msbuild master). 😛

Copy link
Member

Choose a reason for hiding this comment

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

MSBuild master is 16.9 is 5.0.200.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, interesting. We do only build 5.0.1xx, so the logic still applies, but thanks for the correction.

@crummel
Copy link
Contributor

crummel commented Feb 18, 2021

@dseefeld will be looking into this in relation to the 6.0 preview1 PR: #2044.

@crummel
Copy link
Contributor

crummel commented Mar 4, 2021

Triage: superseded by #2044.

@crummel crummel closed this Mar 4, 2021
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.

4 participants