Skip to content

Upgrade MSBuildLocator to version 1.6.1 - #1948

Merged
mmitche merged 6 commits into
mainfrom
dev/jorobich/upgrade-msbuildlocator
Sep 12, 2023
Merged

Upgrade MSBuildLocator to version 1.6.1 #1948
mmitche merged 6 commits into
mainfrom
dev/jorobich/upgrade-msbuildlocator

Conversation

@JoeRobich

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@mthalman

Copy link
Copy Markdown
Member

Looks like there are still failures for aspnetcore and efcore.

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

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

@dotnet-policy-service rerun

@mthalman

Copy link
Copy Markdown
Member

@mmitche - Can you help with this CLA issue?

@mmitche

mmitche commented Sep 12, 2023

Copy link
Copy Markdown
Member

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

4 participants