Skip to content

Conversation

atykhyy
Copy link

@atykhyy atykhyy commented Oct 19, 2023

ILRepack and ILRepackTask currently only copy sequence points and custom debug information into the merged pdb. This makes for a less than ideal debugging experience as local variables are not visible in the debugger. This PR adds a bit of code to copy debug scopes and imports to the merged pdb.

KirillOsenkov added a commit to gluck/il-repack that referenced this pull request Dec 28, 2023
KirillOsenkov added a commit to gluck/il-repack that referenced this pull request Dec 29, 2023
Read embedded pdbs

Copy debug information for scopes and imports
Taken from Alexx999#3

Make the output assembly MVID deterministic

Fix test locking dlls
Add .ToArray() to flush the Linq expression so that it is not evaluated more than once.

Better error if the pdb doesn't exist

Use cecil from https://github.com/KirillOsenkov/cecil/tree/ilrepack
KirillOsenkov added a commit to gluck/il-repack that referenced this pull request Dec 29, 2023
Update Cecil submodule to latest

Read embedded pdbs

Copy debug information for scopes and imports
Taken from Alexx999#3

Make the output assembly MVID deterministic

Fix test locking dlls
Add .ToArray() to flush the Linq expression so that it is not evaluated more than once.

Better error if the pdb doesn't exist

Use cecil from https://github.com/KirillOsenkov/cecil/tree/ilrepack
KirillOsenkov added a commit to gluck/il-repack that referenced this pull request Dec 29, 2023
Read embedded pdbs

Copy debug information for scopes and imports
Taken from Alexx999#3

Make the output assembly MVID deterministic

Fix test locking dlls
Add .ToArray() to flush the Linq expression so that it is not evaluated more than once.

Better error if the pdb doesn't exist

Use cecil from https://github.com/KirillOsenkov/cecil/tree/ilrepack
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