Let runtime supply the portable RID graph

[ViktorHofer]

Contributes to #35750
Related: dotnet/installer#17760

That deleted build task got moved into runtime: https://github.com/dotnet/runtime/blob/faf65649cc5948884e40825889016808b5e36629/src/libraries/Microsoft.NETCore.Platforms/src/Microsoft.NETCore.Platforms.csproj#L52-L60

cc @tmds

[dsplaisted]

It looks like tests are failing because the portable RuntimeIdentifier graph can't be found. Probably the code that creates stage 2 needs to be updated to include that file.

[ViktorHofer]

It looks like tests are failing because the portable RuntimeIdentifier graph can't be found. Probably the code that creates stage 2 needs to be updated to include that file.

Any idea how to do that? The portable and non portable RID graphs are directly flowing from runtime -> installer and bundled into the SDK there.

[ViktorHofer]

@dsplaisted can you please help with this change? I'm currently quite busy with the VMR work and I think we should finish this work before .NET 9 Preview 1.

[kasperk81]

sdk/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.CrossTargeting.targets

Line 119 in f2aca45

<RuntimeIdentifierGraphPath>$([System.IO.Path]::GetDirectoryName($(BundledRuntimeIdentifierGraphFile)))/PortableRuntimeIdentifierGraph.json</RuntimeIdentifierGraphPath>

sdk/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.targets

Line 64 in f2aca45

<RuntimeIdentifierGraphPath Condition="'$(UseRidGraph)' != 'true'">$([System.IO.Path]::GetDirectoryName($(BundledRuntimeIdentifierGraphFile)))/PortableRuntimeIdentifierGraph.json</RuntimeIdentifierGraphPath>

these two needed?

[ViktorHofer]

these two needed?

Those properties in the sdk are needed, yes.

[xen0n]

Hi, apparently this change broke this doc page by accident? As the PortableRuntimeIdentifierGraph.json was removed here.

[akoeplinger]

Thanks, we need to fix the doc page.

[ViktorHofer]

Submitted dotnet/docs#41729