Skip to content

Surface native sourcelink on CoreCLR native PDBs for easier dump debugging #66677

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 17, 2022

Conversation

hoyosjs
Copy link
Member

@hoyosjs hoyosjs commented Mar 15, 2022

No description provided.

@ghost ghost assigned hoyosjs Mar 15, 2022
@ghost
Copy link

ghost commented Mar 15, 2022

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: hoyosjs
Assignees: hoyosjs
Labels:

area-Infrastructure-coreclr

Milestone: -

@hoyosjs hoyosjs force-pushed the juhoyosa/native-sourcelink branch from 0d31790 to 8b0447a Compare March 17, 2022 00:35
@hoyosjs hoyosjs marked this pull request as ready for review March 17, 2022 00:35
@hoyosjs
Copy link
Member Author

hoyosjs commented Mar 17, 2022

@jkoritzinsky can you please take a look at this? Small change - but means changes from CI and releases would automatically pick the right source. It works out of the box in VS. There's a pending fix for WinDBG support.

@hoyosjs
Copy link
Member Author

hoyosjs commented Mar 17, 2022

Failure is #66755

@hoyosjs hoyosjs merged commit 200a11c into dotnet:main Mar 17, 2022
@hoyosjs hoyosjs deleted the juhoyosa/native-sourcelink branch March 17, 2022 18:05
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
…gging (dotnet#66677)

* Use managed sourcelink infrastructure to allow for win native sourcelink
@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants