Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Include host symbols in testhost folder - #122999

#122999
Merged
elinor-fung merged 1 commit into
dotnet:maindotnet/runtime:mainfrom
elinor-fung:testhost-host-symbolselinor-fung/runtime:testhost-host-symbolsCopy head branch name to clipboard
Jan 8, 2026
Merged

Include host symbols in testhost folder#122999
elinor-fung merged 1 commit into
dotnet:maindotnet/runtime:mainfrom
elinor-fung:testhost-host-symbolselinor-fung/runtime:testhost-host-symbolsCopy head branch name to clipboard

Conversation

@elinor-fung

Copy link
Copy Markdown
Member

Copy symbols for dotnet, hostfxr, and hostpolicy when creating the testhost folder.

@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.

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 pull request modifies the build process to include symbol files (PDB files on Windows, .pdb/.dwarf on other platforms) for the host components (dotnet executable, hostfxr, and hostpolicy) when creating the testhost folder. The changes replace the previous single-path symbol file references with glob patterns that search both the main binary directory and a PDB subdirectory.

Key Changes:

  • Replaced specific symbol file references with wildcard-based ItemGroups that search multiple directories
  • Updated Copy tasks to include the new symbol ItemGroups alongside their respective host component binaries
  • Modified Exists conditions from checking the entire ItemGroup to checking individual items with '%(Identity)'

Comment thread src/libraries/externals.csproj
Comment thread src/libraries/externals.csproj
Comment thread src/libraries/externals.csproj
@elinor-fung
elinor-fung requested a review from a team January 8, 2026 03:59
@elinor-fung
elinor-fung merged commit 494093a into dotnet:main Jan 8, 2026
90 of 94 checks passed
@github-project-automation github-project-automation Bot moved this to Done in AppModel Jan 8, 2026
@elinor-fung
elinor-fung deleted the testhost-host-symbols branch January 8, 2026 18:32
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.