-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Upgrade to ICU 72.1.0.4 #121202
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?
Upgrade to ICU 72.1.0.4 #121202
Conversation
|
Tagging subscribers to this area: @dotnet/area-system-globalization |
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.
Pull Request Overview
This PR updates the Microsoft.NETCore.Runtime.ICU.Transport dependency from version 11.0.0-alpha.1.25466.1 to 11.0.0-alpha.1.25530.2, along with updating the corresponding commit SHA in the ICU repository.
Key Changes
- Updated ICU transport package version to a newer alpha build
- Updated the corresponding SHA reference to the dotnet/icu repository
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/Version.Details.xml | Updated the ICU transport dependency version and commit SHA |
| eng/Version.Details.props | Updated the MSBuild property for the ICU transport package version |
|
✅ WASI/WASM failure The new ICU 72 headers mark |
|
✅ WBT that use custom ICU files are failing - their custom ICU files were prepared with ICU v 68. WBT checking non-custom ICU files pass locally. I will update the custom files soon. Addressed by f2e1a67. |
|
linker tests: details: more details when we log the error in
System.Runtime.Tests log |
|
dotnet/icu#686 shipped incorrect prebuilts. They have a bit different size than the ones shipped in icu.transport packages. Static libs were created correctly. That mistake caused
Size diff can be explained by #55637. Not sure yet why the prebuilt file fails and icu.transport does not fail the tests. |
This upstream's approach might be helpful: unicode-org/icu#2629 |
Connected to dotnet/icu#686.
In case of failure, analyze additional changes in #93756.