[natemcmaster]

Part of dotnet/aspnetcore#9720

• React to splitting of Microsoft.NETCore.App into targeting and runtime packs.
• Update to preview6 SDK

cc @dagood

[natemcmaster]

cc @dsplaisted @dagood - is PackagesToReference still needed? I see it's still in core-sdk, but was unclear if there is anything in the package graph of NETCore.App that we need.

[dagood]

I don't know, maybe @nguerrera is also familiar with KnownFrameworkReferences?

[dsplaisted]

We are planning to get rid of it, but it may still be needed for now depending on your scenario. The main thing it does now is bring in the Microsoft.NETCore.Platforms package, which is needed if NuGet needs to understand RuntimeIdentifiers.

FYI, here's the spec for the NuGet feature which will let us drop the package dependency: https://github.com/NuGet/Client.Engineering/pull/96

[natemcmaster]

Good to know. This repo doesn't have any projects using RuntimeIdentifier(s), so I think we're okay.

[dougbu]

@natemcmaster please let us know once you've resolved conflicts and the build cleans up

[natemcmaster]

I'm confused about how my proposed changes resulted in this error:

src\Caching\Abstractions\src\Microsoft.Extensions.Caching.Abstractions.csproj(0,0): error NU1101: Unable to find package Microsoft.DotNet.GenAPI. No packages exist with this id in source(s): https://dotnet.myget.org/F/roslyn/api/v3/index.json, https://dotnet.myget.org/F/sourcelink/api/v3/index.json, https://www.myget.org/F/xunit/api/v3/index.json, nuget.org

I didn't touch the list of restore sources or the package ref to Microsoft.DotNet.GenAPI.

[natemcmaster]

Updating the SDK might have changed the implicit restore feeds. Adding dotnetfeed back to restore sources solved the issue.

[jkotalik]

We pretty much reviewed this together. Had a few feedback comment that were already addressed.