Skip to content
This repository was archived by the owner on Feb 12, 2023. It is now read-only.

Properly detect when .pdb file use the new Portable PDB format #177

Merged
merged 1 commit into from
Sep 11, 2017

Conversation

jairbubbles
Copy link
Contributor

@jairbubbles jairbubbles commented Sep 10, 2017

Rather than throwing an exception we log a warning asking the user to use SourceLink:

image

The process still returns 0 in that case and will not break builds when people switch to new format.

If we do want to return an error, we should have specific error codes for each error cases.

@GeertvanHorrik
Copy link
Contributor

Very nice, thanks for the contribution!

@GeertvanHorrik GeertvanHorrik merged commit d1f2ecf into GitTools:develop Sep 11, 2017
@GeertvanHorrik GeertvanHorrik added this to the 3.1.0 milestone Sep 11, 2017
@jairbubbles jairbubbles deleted the portablePdb branch April 7, 2018 09:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants