-
Notifications
You must be signed in to change notification settings - Fork 323
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
Add portable pdb support for net46. #856
Conversation
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 can be removed?
} | ||
#else | ||
// We don't support full PDB files with .net core | ||
return new PortableSymbolReader(); |
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.
Log on full pdb received for netcore.
/// </returns> | ||
internal static bool IsPortable(Stream stream) | ||
{ | ||
// First four bytes should be 'BSJB' |
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.
BSJB
came from here
(cherry picked from commit c70c0c8)
* Clear extensions cache after discover/execution complete. (#853) * Clear cache after discover/execution complete. * Sort usings * Fix for issue: 1) #632 2) #844 * nitpick: Spelling * Addressed PR comment * spelling correction * nitpick: function name * Removed unused usings (cherry picked from commit 0eef86b) * Add portable pdb support for net46. (#856) (cherry picked from commit c70c0c8) * Install shared farmework compatible with particular version of CLI (#857) (cherry picked from commit 322bf71)
No description provided.