Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@monojenkins
Copy link
Contributor

Backport #18895

Backport of #19049.

/cc @thaystg

@lambdageek lambdageek merged commit b33fa1d into mono:2019-12 Feb 27, 2020
@akoeplinger akoeplinger changed the title [2019-12] [2020-02][debugger] Enable reading embedded ppdb [2019-12] [debugger] Enable reading embedded ppdb Feb 27, 2020
@akoeplinger
Copy link
Member

We might want to proactively bump XA/XI to this commit to make sure it doesn't cause issues for them.

@Therzok
Copy link
Contributor

Therzok commented Feb 27, 2020

Testing this in VSMac, and it's working as intended for us. We've reduced the number of pdbs in the bundle by quite a few

@Therzok
Copy link
Contributor

Therzok commented Feb 27, 2020

Sooo, from the looks of it, VSMac is lucky and the build infrastructure sets the DebugType property after Microsoft.CSharp.Mono.targets is imported.

@thaystg
Copy link
Contributor

thaystg commented Feb 27, 2020

@akoeplinger , I'll do this.

@thaystg
Copy link
Contributor

thaystg commented Feb 27, 2020

Xamarin-Macios -> dotnet/macios#7998

@thaystg
Copy link
Contributor

thaystg commented Feb 27, 2020

Xamarin-Android -> dotnet/android#4335

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.

Enabling system zlib breaks appdomain remoting

5 participants