Skip to content

Fix unique directory received multi tfm#1808

Merged
SimonCropp merged 2 commits into
mainfrom
fix-unique-directory-received-multi-tfm
Jul 20, 2026
Merged

Fix unique directory received multi tfm#1808
SimonCropp merged 2 commits into
mainfrom
fix-unique-directory-received-multi-tfm

Conversation

@SimonCropp

Copy link
Copy Markdown
Member

No description provided.

@SimonCropp SimonCropp added this to the 31.26.0 milestone Jul 20, 2026
…geting

The directory convention derives both the received and the verified paths from
the verified prefix, so, unlike the file convention, every target framework
shared one received path. Each verification also clears received output on init,
via DeleteFiles for the shared directory and DeleteDirectory in split mode, so
concurrent framework runs would delete and overwrite each others received files.

Scope the received side to the runtime and version when multiple frameworks are
targeted. The verified paths are left alone, so committed snapshots do not move
and accepting is unaffected.

Also make the native AOT test skip with the missing prerequisite named, rather
than reporting a machine level toolchain gap as a wall of publish output.
NewLineTests.TrailingNewlinesRaw manages this file itself, deleting it on entry,
writing each variation it needs, and deleting it again at the end. The committed
copy was never read, and every net10.0 run left the deletion in the working tree.

Untrack it and ignore it, matching the sibling StringWithDifferingNewline file.
@SimonCropp
SimonCropp force-pushed the fix-unique-directory-received-multi-tfm branch from 231e5e8 to 37508fb Compare July 20, 2026 12:58
@SimonCropp
SimonCropp merged commit 05912bc into main Jul 20, 2026
3 of 5 checks passed
@SimonCropp
SimonCropp deleted the fix-unique-directory-received-multi-tfm branch July 20, 2026 13:00
@github-actions
github-actions Bot restored the fix-unique-directory-received-multi-tfm branch July 20, 2026 13:00
This was referenced Jul 20, 2026
This was referenced Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant