Referencing System.Reactive fails the build

[jeromelaban]

I require System.Reactive v5.0.0 in my project, I added the nuget package in my "VewiModels" library and now I get this build error:

1>C:\Program Files\dotnet\sdk\7.0.200\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(448,5): error NETSDK1082: There was no runtime pack for Microsoft.WindowsDesktop.App available for the specified RuntimeIdentifier 'win10-arm'.
1>C:\Program Files\dotnet\sdk\7.0.200\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(448,5): error NETSDK1082: There was no runtime pack for Microsoft.WindowsDesktop.App available for the specified RuntimeIdentifier 'win10-arm-aot'.

I found that adding

<RuntimeIdentifiers>win10-x86;win10-x64;win10-arm64</RuntimeIdentifiers>

[Youssef1313]

I wasn't able to reproduce. If there is a repro project, it would be good to start investigating.

[Youssef1313]

Note: This issue is only reproducible with net5.0-windows10.0.19041 and later. I was using 18362 so that's why I couldn't reproduce earlier.

The problem with System.Reactive is that it brings Microsoft.WindowsDesktop.App framework reference for net5.0-windows10.0.19041

The default runtime identifiers for WinUI are: https://github.com/microsoft/WindowsAppSDK/blob/ccc85853cd9d8305194ce0330ad384b5899752d7/dev/MRTCore/packaging/MrtCore.PriGen.targets#L122

and the supported runtime identifiers for Microsoft.WindowsDesktop.App are:

https://github.com/dotnet/installer/blob/0b8f0187eb580aede7dafcb5812e30884150d746/src/redist/targets/GenerateBundledVersions.targets#L271-L274

A possible workaround is to add <DisableTransitiveFrameworkReferences>true</DisableTransitiveFrameworkReferences> to csproj.

[igiona]

DisableTransitiveFrameworkReferences>true

The mentioned workaround works well for me 👍
Thanks for sharing!

I definitely prefer it above this other one I found:
<RuntimeIdentifiers Condition="'$(TargetFramework)'=='net7.0-windows10.0.19041'">win10-x86;win10-x64;win10-arm64</RuntimeIdentifiers>

[Youssef1313]

Happy it worked!
Yes it should be safer than changing RuntimeIdentifiers.

Given that:

• this is not an Uno bug, but something System.Reactive team should fix on their end (and they are already aware of the issue)
• there is a working workaround

I'm going to close this issue.

[igiona]

@Youssef1313 are you aware of a public issue at System.Reactive that can be tracked?

[Youssef1313]

dotnet/reactive#1461 https://github.com/dotnet/reactive/blob/main/Rx.NET/Documentation/Rx-Roadmap-2023.md#unnecessary-bloat-with-certain-build-techniques

GitHub

reactive/Rx-Roadmap-2023.md at main · dotnet/reactive

The Reactive Extensions for .NET. Contribute to dotnet/reactive development by creating an account on GitHub.