-
Notifications
You must be signed in to change notification settings - Fork 110
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
React to breaking RID change in .NET 8 Preview 6 #543
Comments
This is actually a bug in the warning. It should not be flagging those two. dotnet/sdk#33584 should be fixing it. |
It seems to be a direct copy. I think alpine-* should just go away: SQLitePCL.raw/gen_lib_nuspecs/Program.cs Lines 409 to 417 in 760d88f
|
So the musl part of this change looks straightforward. I just need to delete those alpine RIDs. The change from And I'm even less clear about whether any other changes are needed w.r.t. MAUI stuff. But none of this is difficult once I'm sure I'm doing it right. :-) |
The only change needed should be removing the alpine RIDs. The other ones that were flagged (brower, maccatalyst) were flagged incorrectly - it was a bug in the warning itself. |
@elinor-fung The Alpine RIDs were removed from the package, but we're still getting an error in dotnet/efcore:
We're using a very recent build of the SDK:
The SQLitePCLRaw.lib.e_sqlite3 package now has the following:
|
That looks like dotnet/sdk#34003 (based on logs from https://dev.azure.com/dnceng-public/public/_build/results?buildId=369044&view=results). |
@ericsink I am getting this error when compiling a class library with SDK 8.0.204. which references Microsoft.EntityFrameworkCore.Sqlite.Core 8.0.4
|
There's a breaking change to the .NET SDK coming in .NET 8 Preview 6 (Not preview 5 like the docs indicate @gewarren) that makes any app depending on SQLitePCLRaw.core to output a warning like the following at runtime:
You need to install the latest preview 6 SDK which isn't yet released to repo. In the meantime, you can install the nightlies from https://github.com/dotnet/installer#table
Once you do, you can create a basic console app using something like
dotnet new console
, make sure you're using a preview6 SDK with a global.json, replace the csproj with:If you run or publish the app, you see the following warning. Fortunately, this does not stop the app from running successfully, but it adds noise to the output of any app depending on the package directly or transitively.
My understanding is that RIDs like
alpine-arm
should be replaced with justlinux-musl-arm
which already exists. Similarly,browser-wasm
should be replaced with justwasm
. I'm not sure about themaccatalyst
targets. I assume theosx
binaries are incompatible since they come from a different build artifact in the nuspec.https://learn.microsoft.com/en-us/dotnet/core/compatibility/deployment/8.0/rid-asset-list
@gewarren @elinor-fung @richlander I think the breaking change announcement might need to go into a lot more detail about what's allowed. It wasn't clear to me that
wasm
is probably the intended replacement forbrowser-wasm
until looking at dotnet/runtime#83246. WASM isn't mentioned anywhere in the breaking change announcement. Neither are the full list of OS's, libc versions, and processor architectures.@davidortinau Do you know what the plan is for libraries that target maui-supported RIDs like
maccatalys
,android
andios
?@roji Heads up in case you start seeing this in Microsoft.Data.Sqlite.Core
The text was updated successfully, but these errors were encountered: