PDBs for MRT Core are missing source server information #4633
Labels
area-Infrastructure
Build, test, source layout, package construction (TODO: move to Deployment, DeveloperTools)
area-MRTCore
ResourceManager
fix-released
The fix has been in a release (experimental, preview, stable, or servicing).
Status: Fixed internally
The issue has been fixed but has not yet shipped in a release.
team-Markup
Milestone
Comments
|
There's a very real possibility that the presence of a |
codendone
added
area-Deployment
DrusTheAxe
added
area-Infrastructure
evelynwu-msft
added a commit
that referenced
this issue
Aug 21, 2024
…al AzDO mirror (#4666) This repository is mirrored onto an internal Azure DevOps repository that is used for building Windows App SDK releases. This has the unfortunate side effect of embedding into the PDB symbols Source Link information that points at the internal AzDO repo rather than this public Github repo. This change addresses that by adding a custom target that corrects the repo URL that Source Link uses. Fixes #4633.
evelynwu-msft
added a commit
that referenced
this issue
Aug 21, 2024
…al AzDO mirror (#4666) This repository is mirrored onto an internal Azure DevOps repository that is used for building Windows App SDK releases. This has the unfortunate side effect of embedding into the PDB symbols Source Link information that points at the internal AzDO repo rather than this public Github repo. This change addresses that by adding a custom target that corrects the repo URL that Source Link uses. Fixes #4633. (cherry picked from commit a820af7)
bpulliam
pushed a commit
that referenced
this issue
Aug 21, 2024
…points to Github rather than internal AzDO mirror (#4670) This repository is mirrored onto an internal Azure DevOps repository that is used for building Windows App SDK releases. This has the unfortunate side effect of embedding into the PDB symbols Source Link information that points at the internal AzDO repo rather than this public Github repo. This change addresses that by adding a custom target that corrects the repo URL that Source Link uses. Fixes #4633. (cherry picked from commit a820af7)
codendone
added
Status: Fixed internally
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The PDBs for MRT Core (both
Microsoft.Windows.ApplicationModel.Resources.dllandMRM.dll) are missing source server information for the source files. As a result, when debugging MRT Core code developers are required to manually obtain the matching source code (e.g. cloning this repository) and point their debugger at it. Additionally, the internal AzDO repo is what's pointed to by the PDB rather than this Github repo, so even if the expected source server entries were present non-Microsoft devs would still be unable to take advantage of that.Steps to reproduce the bug
c:\Program Files (x86)\Windows Kits\10\Debuggers\x64\srcsrv>symchk /v "C:\Program Files\WindowsApps\Microsoft.WindowsAppRuntime.1.6-preview1_6000.219.2254.0_x64__8wekyb3d8bbwe\Microsoft.Windows.ApplicationModel.Resources.dll" /s https://msdl.microsoft.com/download/symbols /oc c:\symbols\)srcsrvstream from the PDBs downloaded in step 2:c:\Program Files (x86)\Windows Kits\10\Debuggers\x64\srcsrv>pdbstr -r -p:"c:\symbols\Microsoft.Windows.ApplicationModel.Resources.pdb" -i:c:\Microsoft.Windows.ApplicationModel.Resources.txt -s:srcsrvExpected behavior
The
srcsrvstreams should contain source information entries for every source file in https://github.com/microsoft/WindowsAppSDK/tree/main/dev/MRTCore/mrt/.Screenshots
No response
NuGet package version
Windows App SDK 1.6 Experimental 2: 1.6.240701003-experimental2
Packaging type
No response
Windows version
No response
IDE
No response
Additional context
The actual
srcsrvstreams for Microsoft.Windows.ApplicationModel.Resources.dll and MRM.dll.For comparison, this is the stream for Microsoft.WindowsAppRuntime.Insights.Resource.dll which contains the expected information.
The text was updated successfully, but these errors were encountered: