-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Upgrade dotnet SDK to 3.0-preview3 #36859
Conversation
0742d9b to
2141989
Compare
|
Validation official PR: https://dnceng.visualstudio.com/internal/_build/results?buildId=155913 |
safern
left a comment
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.
LGTM
|
@ericstj seems like duplicate types issues with netstandard2.0 vs netstandard2.1. Can you please take a look as you are the expert here? |
|
Yes there are currently duplicate types. @terrajobst and @wtgodbe likely need to add new shims to the targeting pack (or we need to ship new versions of all these packages targeting ns2.1) to resolve it. What did you do to trigger running the package tests against .NETStandard 2.1? |
Nothing, that's by default. |
|
I see, we expand to any compatible frameworks and you brought in the new NuGet that knows about the derived frameworks. We can add suppression for the duplicate types for now until this can be fixed. Let's capture the list of duplicate types and put it in a NS2.1 suppression props. |
|
Some of these are different: /cc @tannergooding I'll share a type-duplicate list for the ns2.1 stuff. @wtgodbe needs that anyway. |
|
Here are the duplicates I found throughout the tests. This doesn't cover the netcoreapp3.0-specific failures. For those it looks like we might need a different workaround.
|
|
For NETCoreApp3.0 I suspect the 3.0 SDK is no longer honoring RuntimeFrameworkVersion. We need to find out how to override the version of refs it is using. |
SDK is ignoring RuntimeFrameworkVersion for ref packs. Set TargetingPackVersion on KnownFrameworkReference instead. dotnet/sdk#3129
* Upgrade SDK to 3.0 * Remove asp and desktop framework references * Manual darc update from build '20190415.1' * Suppress type overlap with ns2.1 * Make SDK use output of core-setup for both ref and runtime SDK is ignoring RuntimeFrameworkVersion for ref packs. Set TargetingPackVersion on KnownFrameworkReference instead. dotnet/sdk#3129 Commit migrated from dotnet/corefx@66bcc2d
Upgrade to 3.0.
Requires dotnet/arcade#2517
Fixes https://github.com/dotnet/corefx/issues/34498