-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RIDs for Tizen8.0 #94499
Add RIDs for Tizen8.0 #94499
Conversation
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue Detailsnull
|
Eventually the goal is to remove this (tracked by #90000). I don't understand why runtime/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj Line 15 in 173eb1f
but I assume that's a good reason to keep updating this. For Linux, these updates happen automagically during source-build. |
Are we sure we need these Tizen RID updates just for shared framework deps.json file? Would it make sense to require the portable RID graph (which already is the default) when interacting with Tizen RIDs when targeting .NET 8 / .NET 9? cc @jkotas @agocke From the other thread (cc @elinor-fung):
IMO we should start adding a warning in .NET 9 and then remove the switch with .NET 10. Otherwise we will never be able to stop updating and removing the legacy RID graph. |
If this property is just for packing the old graph. Then I fully agree: we don't need to add any more rids and consider the legacy graph unsupported for platforms not yet included. |
@vitek-karas mentioned here, that the Microsoft.NETCore.App.deps.json includes the full (legacy) RID graph to make that opt-out switch work. |
The idea was that we do need to keep the big RID graph in |
Based on that, I'm closing this PR. |
No description provided.