Skip to content

[rel/18.7] Remove stale Microsoft.Extensions.FileSystemGlobbing binding redirect - #16084

Merged
Amaury Levé (Evangelink) merged 1 commit into
microsoft:rel/18.7from
Evangelink:dev/amauryleve/remove-stale-filesystemglobbing-redirect-rel-18.7
Jun 2, 2026
Merged

[rel/18.7] Remove stale Microsoft.Extensions.FileSystemGlobbing binding redirect#16084
Amaury Levé (Evangelink) merged 1 commit into
microsoft:rel/18.7from
Evangelink:dev/amauryleve/remove-stale-filesystemglobbing-redirect-rel-18.7

Conversation

@Evangelink

Copy link
Copy Markdown
Member

Backport of #16082 to rel/18.7.

What

Remove the stale Microsoft.Extensions.FileSystemGlobbing <bindingRedirect> from src/testhost.x86/app.config and src/datacollector/app.config (kept in src/vstest.console/app.config).

Why

Neither testhost.x86.exe nor datacollector.exe references the assembly, and the DLL doesn't ship beside them in the Microsoft.TestPlatform.CLI nupkg's TestHostNetFramework folder. Same anti-pattern as the DiagnosticSource fix in #15776.

The active failure is on the rel/18.6 VMR build (see #16083); rel/18.7 carries the same stale redirect and the same missing DotNetBuild != 'true' skip on _VerifyNuGetPackages, so this backport keeps the branches consistent and prevents the same VMR failure if rel/18.7 packs in that context.

…ng redirect

Cherry-pick of the main fix (PR microsoft#16082) for consistency with rel/18.6.

testhost.x86 and datacollector neither reference Microsoft.Extensions.FileSystemGlobbing nor ship the DLL beside their EXE in the Microsoft.TestPlatform.CLI nupkg (TestHostNetFramework folder). Same anti-pattern as the DiagnosticSource fix (microsoft#15776).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Evangelink
Amaury Levé (Evangelink) enabled auto-merge (squash) June 2, 2026 11:57
@Evangelink
Amaury Levé (Evangelink) merged commit 770837f into microsoft:rel/18.7 Jun 2, 2026
7 checks passed
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