Skip to content

Revert "[main] Source code updates from dotnet/dotnet"#16744

Merged
mmitche merged 1 commit into
mainfrom
revert-16726-darc-main-b46622d4-f860-4171-9113-5c232204d1f3
Apr 23, 2026
Merged

Revert "[main] Source code updates from dotnet/dotnet"#16744
mmitche merged 1 commit into
mainfrom
revert-16726-darc-main-b46622d4-f860-4171-9113-5c232204d1f3

Conversation

@mmitche
Copy link
Copy Markdown
Member

@mmitche mmitche commented Apr 23, 2026

Reverts #16726

@mmitche mmitche requested a review from a team as a code owner April 23, 2026 20:53
@mmitche mmitche enabled auto-merge (squash) April 23, 2026 20:54
@mmitche mmitche disabled auto-merge April 23, 2026 23:47
@mmitche mmitche merged commit 4c6b947 into main Apr 23, 2026
6 of 9 checks passed
@mmitche mmitche deleted the revert-16726-darc-main-b46622d4-f860-4171-9113-5c232204d1f3 branch April 23, 2026 23:48
lbussell added a commit to dotnet/docker-tools that referenced this pull request May 12, 2026
This PR fixes `docker-tools-imagebuilder-official` failing in `Test
Images` after the Arcade update in #2099.

The root cause is that Arcade stopped writing
`artifacts/toolset/<version>.txt` and now restores the SDK toolset to
`artifacts/toolset/<version>/Build.proj`.

Relevant Arcade history:
- dotnet/arcade#16726 originally introduced the `InitializeToolset`
layout change.
- dotnet/arcade#16744 reverted that source update.
- dotnet/arcade#16753 reintroduced the layout change; this is the change
present in the Arcade build consumed by #2099.

This PR updates `src/run-tests.ps1` to resolve the new `Build.proj`
location first while retaining a legacy marker-file fallback for older
Arcade layouts.

Validation:
- `git clean -ffxd`
- `pwsh -NoLogo -NoProfile -NonInteractive -File ./src/run-tests.ps1`
(478 tests passed)

Fixes #2104

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

2 participants