Skip to content

Conversation

@pjcollins
Copy link
Member

Context: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1421424/

A recent update to Mono.Options.dll (presumably commit c936d09) has
caused a symbol check failure in our latest VS insertion. We can fix
this by updating one of the tools projects that references this package
to copy Mono.Options.pdb to the Output directory. Once the file is in
the right output path we can include it in the XA installers.

Context: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1421424/

A recent update to Mono.Options.dll (presumably commit c936d09) has
caused a symbol check failure in our latest VS insertion.  We can fix
this by updating one of the tools projects that references this package
to copy Mono.Options.pdb to the Output directory.  Once the file is in
the right output path we can include it in the XA installers.
@pjcollins
Copy link
Member Author

Testing this in dotnet/android#6407

@pjcollins pjcollins requested review from jonpryor and jpobst October 20, 2021 17:51
@pjcollins
Copy link
Member Author

Looks like installer creation succeeded so the 4.0 .pbd is getting to the right output dir in XA. I think this should be good to go.

@jonpryor jonpryor merged commit 974ad32 into main Oct 20, 2021
@jonpryor jonpryor deleted the dev/pecolli/mono-options-pdb branch October 20, 2021 21:49
@jpobst
Copy link
Contributor

jpobst commented Oct 21, 2021

I wonder why this started failing.

Neither of the previous versions of Mono.Options NuGets we were using (5.3.0.1 or 6.6.0.161) had .pdb files. The latest one we updated to does (6.12.0.148), but pdb files from NuGets aren't copied to the output directory:

dotnet/sdk#1458

@pjcollins
Copy link
Member Author

I think the previous version of Mono.Options in our installer was old enough that it already had symbols archived from some other insertion, so the particular file was never flagged.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2024
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.

4 participants