-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Target net11.0 / .NET 11 #121853
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
base: main
Are you sure you want to change the base?
Target net11.0 / .NET 11 #121853
Conversation
|
Tagging subscribers to this area: @dotnet/runtime-infrastructure |
|
@ViktorHofer, does it depend on dotnet/dotnet#2271? |
|
Not necessarily but it would be better to get that in first. I included the branding update in this PR (see eng/Versions.props changes). |
Need this property: https://github.com/dotnet/runtime/blob/69310d88a431951addcc65ad093ea59528f80c48/Directory.Build.targets#L56C6-L56C30 |
@am11 good suggestion (as usual) |
It looks to me like something in https://github.com/dotnet/runtime/blob/main/src/mono/wasm/build/WasmApp.Common.targets is failing to publish the runtime bits. Was talking to @steveisok offline and it could be due to no runtime pack being available. It looks like in 10.0 we had 10.0 runtime packs available by the time we were retargeting but we don't have that right now. I'm not certain if this is the problem, but it seems plausible. |
/native/managed/cdac/ -->
|
They are the community supported platforms using bootstrapping mechanism (aka from-scratch build): https://github.com/dotnet/runtime/blob/3e44f568cb24aa68f863d925b67fa5408b734d54/docs/workflow/building/coreclr/cross-building.md#building-coreclr-with-bootstrapping. First pass builds minimal subsets: shared framework and tools, second pass moves them in
They need it to publish packages with executables during the build like Microsoft supported platforms. |
Are they applications that actually ship the runtime? These looked to me like libraries that would depend on something else to bring the runtime. If they don't need runtime packs, they shouldn't restore them (change properties that are making the SDK restore these). If they do need the runtime packs, we need to plumb the live ones. |
|
Yes, all tools which are in linux-arm64 SDK are included in SDKs for those RIDs, e.g. https://github.com/filipnavara/dotnet-riscv/releases/tag/10.0.100. |
I think it might be easier to skip the cdac subsets on the community legs for now and bring it back once we have an 11 sdk and packs. |
|
@steveisok, could you please add |
I moved it out and it still made no difference. I had that in |
--- a/src/native/managed/cdac/Directory.Build.props
+++ b/src/native/managed/cdac/Directory.Build.props
@@ -1,5 +1,11 @@
<Project>
<Import Project="..\Directory.Build.props" />
+
+ <PropertyGroup>
+ <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
+ <AppendTargetFrameworkToOutputPath Condition="'$(BuildingInsideVisualStudio)' == 'true'">true</AppendTargetFrameworkToOutputPath>
+ </PropertyGroup>
+
<ItemGroup>
<AssemblyAttribute Include="System.Diagnostics.CodeAnalysis.ExperimentalAttribute">
<_Parameter1>NETCDAC0001</_Parameter1> |
|
I didn't asked to move it out, I meant to add it in cDAC 🙂 |
Bad use of words - meant the same thing. |
|
I didn't find that change pushed in this PR. Did you tried it locally on some community platform? Could you show the command (or just push to branch so we can see the error in CI). |
I tried it locally on cross building freebsd. Same exact restore error. |
|
I tested this patch am11@1cfe520 and built:
both passed the restore and cDAC package build and later failing at ApiCompat stage. The reason why I was asking you for the command or pushing the change because just like my workflow, runtime CI is also using |
Yep, unrelated: #122171 (comment) |
|
Looking at the patch, I believe @am11's patch is the right solution for the cDAC problem. |
I tried it both locally and I see @lewing pushed a change. Neither work. The |
|
#122258 has sdk branding flow which will make it easier to solve the remaining workload issues here but we will keep both up to date. dotnet/sdk#52031 is in progress for the sdk |
Contributes to #118583
For reference, this was last year's PR for net10.0. #106599