Skip to content

Load Microsoft.DiaSymReader.Native from the runtime - #605

Open
tmat wants to merge 6 commits into
dotnet:mainfrom
tmat:remove-diasymreader-native-package
Open

tmat wants to merge 6 commits into
dotnet:mainfrom
tmat:remove-diasymreader-native-package

Conversation

@tmat

@tmat tmat commented Sep 11, 2026

Copy link
Copy Markdown
Member

The .NET runtime now ships Microsoft.DiaSymReader.Native.*.dll in the shared framework, so the Microsoft.DiaSymReader.Native package reference is no longer needed.

  • Copy the native DLL from the shared framework into test output instead of restoring the NuGet package.

tmat and others added 6 commits September 11, 2026 11:26
The .NET runtime now ships Microsoft.DiaSymReader.Native.*.dll in the shared
framework, so the Microsoft.DiaSymReader.Native package reference is no longer
needed. Load the native binaries from the runtime instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Load the native binaries from the .NET runtime instead of copying them
into test output. .NET tests rely on the runtime load path. .NET Framework
tests call AddDllDirectory with the installed .NET shared-framework directory.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Wrap the test helper in a single NETFRAMEWORK block instead of
scattering ifdefs through APIs that only run on .NET Framework.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
64-bit is the default; no need to set the architecture explicitly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Install the x86 shared framework from global.json tools.runtimes and run
Framework tests on both x64 and x86. Core x86 is omitted until
Microsoft.DiaSymReader passes Guid by pointer. Framework tests locate
Native.x86.dll under .dotnet/x86.

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.

1 participant