Skip to content

Conversation

@JoeRobich
Copy link
Member

This includes several fixes around resolving hostfxr. (See microsoft/MSBuildLocator#230 & microsoft/MSBuildLocator#236)

resolves #1945

This includes several fixes around resolving hostfxr.
@JoeRobich JoeRobich requested a review from sharwell August 30, 2023 17:22
@mthalman mthalman closed this Sep 11, 2023
@mthalman mthalman reopened this Sep 11, 2023
@mthalman
Copy link
Member

I updated the version of source-build-externals because the previous version contained 1.5.5 of MSBuildLocator. So this update to 1.6.1 cause a source-build prebuilt error to occur. The latest version of source-build-externals has version 1.6.1 so this should resolve the error.

@mthalman
Copy link
Member

@JoeRobich or @sharwell, could you investigate the build errors? This change is needed to unblock code flow for the sdk: dotnet/installer#17319.

@JoeRobich
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@mthalman
Copy link
Member

Looks like there are still failures for aspnetcore and efcore.

_targetSolution: "format.sln"
_branchName: "main"
_sha: "1ec992c6038af9d42499d3ac88c3fd65d0c2f6ed"
_sha: "452f6263277533da0af2b557003256d98fbecc27"
Copy link
Contributor

Choose a reason for hiding this comment

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

❔ Why update all of these?

Copy link
Member Author

Choose a reason for hiding this comment

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

Habit. Although I see these repos are now on some very preview sdk versions

Copy link
Contributor

Choose a reason for hiding this comment

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

If updating the dependency broke formatting for the previous SHA, doesn't that mean some users could see dotnet-format break for the same change?

@JoeRobich JoeRobich changed the title Upgrade MSBuildLocator to version 1.6.1 Upgrade MSBuildLocator to version 1.6.1 Sep 12, 2023
@sharwell sharwell marked this pull request as draft September 12, 2023 14:12
@sharwell sharwell marked this pull request as ready for review September 12, 2023 14:12
@sharwell sharwell marked this pull request as draft September 12, 2023 14:50
@sharwell sharwell marked this pull request as ready for review September 12, 2023 14:50
@sharwell sharwell marked this pull request as draft September 12, 2023 15:52
@sharwell sharwell marked this pull request as ready for review September 12, 2023 15:52
@mthalman
Copy link
Member

@dotnet-policy-service rerun

@mthalman
Copy link
Member

@mmitche - Can you help with this CLA issue?

@mmitche
Copy link
Member

mmitche commented Sep 12, 2023

Nope. But I can merge around it!

@mmitche mmitche merged commit 389e90e into main Sep 12, 2023
@ghost ghost added this to the Next milestone Sep 12, 2023
@sharwell sharwell deleted the dev/jorobich/upgrade-msbuildlocator branch September 12, 2023 18:12
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.

dotnet-format 8.0 rc1 is failing on Alpine 3.17

5 participants