[main] Update dependencies from dotnet/runtime #31421
Conversation
![dotnet-maestro[bot]](https://avatars.githubusercontent.com/in/16706?s=60&v=4){kind=small}
…0806.6 Microsoft.Extensions.Caching.Memory , Microsoft.Extensions.Configuration , Microsoft.Extensions.Configuration.Abstractions , Microsoft.Extensions.Configuration.EnvironmentVariables , Microsoft.Extensions.Configuration.Json , Microsoft.Extensions.DependencyInjection , Microsoft.Extensions.DependencyModel , Microsoft.Extensions.HostFactoryResolver.Sources , Microsoft.Extensions.Logging , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.BrowserDebugHost.Transport , System.Text.Json From Version 8.0.0-rc.1.23402.2 -> To Version 8.0.0-rc.1.23406.6
ghost
added
|
ptal @dotnet/interop-contrib |
…0814.1 Microsoft.Extensions.Caching.Memory , Microsoft.Extensions.Configuration , Microsoft.Extensions.Configuration.Abstractions , Microsoft.Extensions.Configuration.EnvironmentVariables , Microsoft.Extensions.Configuration.Json , Microsoft.Extensions.DependencyInjection , Microsoft.Extensions.DependencyModel , Microsoft.Extensions.HostFactoryResolver.Sources , Microsoft.Extensions.Logging , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.BrowserDebugHost.Transport , System.Text.Json From Version 8.0.0-rc.1.23402.2 -> To Version 8.0.0-rc.1.23414.1
|
@dotnet/dnr-codeflow @dotnet/efteam I don't know who decides on the compiler version but I believe this needs to be resolved for a coherent build? cc @mmitche |
For Microsoft's builds, there is no coherency requirement for the compiler/sdk. For source build (of which efcore is not involved), the installer repo determines what gets used. This repo should upgrade its SDK to match runtime's. |
|
@mmitche this repo is already using a later SDK than runtime, and downgrading to match doesn't help. efcore: 8.0.100-rc.1.23381.2 Changing |
|
Sorry, it's the version of roslyn in the SDK that matters. Runtime upgraded roslyn outside of the SDK: https://github.com/dotnet/runtime/blame/main/eng/Versions.props#L43 However, I would try a newer SDK, e.g. 8.0.100-rc.1.23416.11 which should carry a new enough roslyn. |
|
8.0.100-rc.1.23416.11 didn't help, it just changed the error message from recommending 'preview' to '12.0', and updating to version 12 changed the error message to say use 'in'. Updating the SDK also introduced other Aot errors so I won't go there right now, just changing 'ref' to 'in' seems to work and be required regardless. |
@tannergooding For review of the ref->in change. |
yeah this was dotnet/runtime#89736 |
dotnet-maestro
bot
deleted the
darc-main-49e9ae75-dcfc-4eca-81b1-d8c7924634fd
branch
This pull request updates the following dependencies
From https://github.com/dotnet/runtime