-
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
[main] Update dependencies from dotnet/cecil, dotnet/hotreload-utils, dotnet/icu, dotnet/runtime, dotnet/runtime-assets, dotnet/sdk, dotnet/xharness #97640
Conversation
…0129.2 Microsoft.DotNet.ILCompiler , Microsoft.NET.Sdk.IL , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.ILAsm , runtime.native.System.IO.Ports , System.Text.Json From Version 9.0.0-alpha.1.24072.1 -> To Version 9.0.0-preview.2.24079.2
Microsoft.DotNet.ApiCompat.Task From Version 9.0.100-alpha.1.24072.3 -> To Version 9.0.100-preview.2.24078.5
Microsoft.NETCore.Runtime.ICU.Transport From Version 9.0.0-preview.2.24072.3 -> To Version 9.0.0-preview.2.24079.1
…40130.1 Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit From Version 9.0.0-prerelease.24077.1 -> To Version 9.0.0-prerelease.24080.1
…ld 20240130.1 Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData From Version 9.0.0-beta.24072.1 -> To Version 9.0.0-beta.24080.1
…ild 20240129.1 Microsoft.DotNet.HotReload.Utils.Generator.BuildTool From Version 9.0.0-alpha.0.24072.1 -> To Version 9.0.0-alpha.0.24079.1
…29.1 Microsoft.DotNet.Cecil From Version 0.11.4-alpha.24065.1 -> To Version 0.11.4-alpha.24079.1
Microsoft.NETCore.Runtime.ICU.Transport From Version 9.0.0-preview.2.24072.3 -> To Version 9.0.0-preview.2.24079.1
…40130.1 Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit From Version 9.0.0-prerelease.24077.1 -> To Version 9.0.0-prerelease.24080.1
…ld 20240130.1 Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData From Version 9.0.0-beta.24072.1 -> To Version 9.0.0-beta.24080.1
…ild 20240129.1 Microsoft.DotNet.HotReload.Utils.Generator.BuildTool From Version 9.0.0-alpha.0.24072.1 -> To Version 9.0.0-alpha.0.24079.1
…30.1 Microsoft.DotNet.Cecil From Version 0.11.4-alpha.24065.1 -> To Version 0.11.4-alpha.24080.1
Microsoft.NETCore.Runtime.ICU.Transport From Version 9.0.0-preview.2.24072.3 -> To Version 9.0.0-preview.2.24079.1
…40130.1 Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit From Version 9.0.0-prerelease.24077.1 -> To Version 9.0.0-prerelease.24080.1
…ld 20240130.1 Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData From Version 9.0.0-beta.24072.1 -> To Version 9.0.0-beta.24080.1
…ild 20240205.1 Microsoft.DotNet.HotReload.Utils.Generator.BuildTool From Version 9.0.0-alpha.0.24072.1 -> To Version 9.0.0-alpha.0.24105.1
…05.1 Microsoft.SourceBuild.Intermediate.cecil , Microsoft.DotNet.Cecil From Version 0.11.4-alpha.24065.1 -> To Version 0.11.4-alpha.24105.1
Microsoft.SourceBuild.Intermediate.sdk , Microsoft.DotNet.ApiCompat.Task From Version 9.0.100-alpha.1.24072.3 -> To Version 9.0.100-preview.2.24112.1
src/tasks/Microsoft.NET.WebAssembly.Webcil/Microsoft.NET.WebAssembly.Webcil.csproj
Outdated
Show resolved
Hide resolved
eng/Versions.props
Outdated
<SystemReflectionMetadataToolsetVersion>8.0.0</SystemReflectionMetadataToolsetVersion> | ||
<SystemReflectionMetadataLoadContextToolsetVersion>8.0.0</SystemReflectionMetadataLoadContextToolsetVersion> | ||
<!-- when building from source we need to use the current version of MetadataLoadContext --> | ||
<SystemReflectionMetadataLoadContextToolsetVersion Condition="'$(DotNetBuildFromSource)' == 'true'">$(SystemReflectionMetadataLoadContextVersion)</SystemReflectionMetadataLoadContextToolsetVersion> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I question if this is going to work correctly in the product source build. The source build versions.props gets imported after this. The SystemReflectionMetadataLoadContextToolsetVersion is not going to be set to the correct SB version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @mmitche
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MichaelSimons is correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Viktor suggested setting SystemReflectionMetadataLoadContextToolsetVersion in Directory.Build.targets so we get the correct version in source build, pushed that.
I haven't look at this closely but are these changes going to fix this issue related to WasmAppBuilder: dotnet/installer#18616 (comment) |
@mthalman yes |
This pull request updates the following dependencies
From https://github.com/dotnet/runtime
From https://github.com/dotnet/sdk
From https://github.com/dotnet/icu
From https://github.com/dotnet/xharness
From https://github.com/dotnet/runtime-assets
From https://github.com/dotnet/hotreload-utils
From https://github.com/dotnet/cecil