Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Missing debugging symbols for System.Private.* stuff #941

Closed
MichalStrehovsky opened this issue Feb 25, 2016 · 3 comments
Closed

Missing debugging symbols for System.Private.* stuff #941

MichalStrehovsky opened this issue Feb 25, 2016 · 3 comments

Comments

@MichalStrehovsky
Copy link
Member

After running build.cmd, the location at bin\Product\Windows_NT.x64.Debug\.nuget\publish1\sdk only has DLL files, but PDBs are missing.

RyuJIT.rsp picks up System.Private.* libraries from this location, but ILC can't find the symbols.

I gave myself 5 mins to figure this out, but I couldn't find the spot where this directory is populated :/

@MichalStrehovsky
Copy link
Member Author

@schellap If you can give me some pointers, I can look into it. Or you're also welcome to fix this :)

It's possible to work around this by manually copying the PDBs into that location (which is what I've been doing in the past few days).

@schellap
Copy link

You can copy the pdbs into this folder temporarily. May be as a build step also. I can TAL tomorrow.

Purely CLI way would be to make symbol packages and restore those as well as dependencies in the stage1 project.json. With the current publish1 folder, rather than xcopy based binplace, we get the value of just using dotnet to validate our stuff which is useful for maintaining contracts and shipping. But I don't see much value purely for internal testing to produce symbol packages.

@jkotas
Copy link
Member

jkotas commented Feb 26, 2016

For our internal debugging in VS, we should fix up ryujit.rsp (using powershell or something similar) to have the right paths, so that F5 just works (even for changes in System.Private.CoreLib). Right now, I am working around this by editing paths in ryujit.rsp to point to \corert\bin\Product\Windows_NT.x64.Debug.

#907 is tracking the fix for the end user case.

MichalStrehovsky added a commit to MichalStrehovsky/corert that referenced this issue Mar 28, 2016
This replaces references to assemblies under publish1\sdk to point to
the live built ones.

Fixes dotnet#941.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants