-
Notifications
You must be signed in to change notification settings - Fork 842
Embed PDBs in built binary #1529
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
Conversation
|
"This branch has conflicts that must be resolved"? |
| //--------------------------------------------------------------------- | ||
| // Portable PDB Writer | ||
| //--------------------------------------------------------------------- | ||
| type BinaryChunk = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Put this in ilwrite.fsi. It is not pdb specific.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reverts commit 992aa7a.
| --resource:<resinfo> Embed the specified managed resource | ||
| --linkresource:<resinfo> Link the specified resource to this assembly | ||
| where the resinfo format is <file>[,<string | ||
| name>[,public|private]] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the help text still look OK visually when printed on an 80-column console window? thanks
|
@KevinRansom Please add at least one invocation of the compiler that uses |
|
1: I will add a smoke test when I have a machine that build again ... I am paying a steep price for having to install the latest internal VS builds right now. full pdbs. |

This PR embeds compressed PDBs into the built binary.
It adds embedded as an option to --debug
updates command line tests.