Skip to content

Update global.json - #117460

Closed
ViktorHofer wants to merge 4 commits into
mainfrom
ViktorHofer-patch-1
Closed

Update global.json#117460
ViktorHofer wants to merge 4 commits into
mainfrom
ViktorHofer-patch-1

Conversation

@ViktorHofer

Copy link
Copy Markdown
Member

Use the new paths feature and allow newer SDKs when building

https://learn.microsoft.com/en-us/dotnet/core/tools/global-json#paths

Use the new paths feature and allow newer SDKs when building

https://learn.microsoft.com/en-us/dotnet/core/tools/global-json#paths
Copilot AI review requested due to automatic review settings July 9, 2025 13:51

Copilot AI left a comment

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.

Pull Request Overview

This PR enhances the global.json for SDK lookup flexibility by introducing the new paths feature, allowing SDK roll-forward on major versions, and providing a custom error message when the required SDK is missing.

  • Added "paths" array to include local .dotnet folder and $host$ placeholder.
  • Kept "rollForward": "major" to permit newer SDKs.
  • Included "errorMessage" to guide developers to the install script.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@huoyaoyuan

Copy link
Copy Markdown
Member

Can we remove the dotnet.cmd/sh helper when 10.0 releases and the feature is GA?

@ViktorHofer

ViktorHofer commented Jul 9, 2025

Copy link
Copy Markdown
Member Author

Yes, when VS supports it fully :)

@agocke

agocke commented Jul 9, 2025

Copy link
Copy Markdown
Member

Note: won’t help for runtime execution. It’s an sdk only feature. We might want to keep the scripts around if someone needs to run a tool on the latest

@agocke

agocke commented Jul 9, 2025

Copy link
Copy Markdown
Member

(Also ‘dnvm restore’ will just install the sdk locally so you don’t have to worry about this).

@ViktorHofer

Copy link
Copy Markdown
Member Author

Right but I would still want to replace these scripts eventually with the eng/common/dotnet.cmd/sh scripts that we now have.

@akoeplinger

Copy link
Copy Markdown
Member

The WasmBuildTests failures are related

@ViktorHofer
ViktorHofer deleted the ViktorHofer-patch-1 branch September 29, 2025 16:17
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants