-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[mono] Test upgrading to ICU 72.3 #93756
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
Conversation
|
looks like they deprecated some apis 😬 |
|
Same issue found in the previous 72 bump. We're likely filtering out some necessary collation data. This applies to both wasm and ios. |
|
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
| const DotnetSupportLib = { | ||
| $DOTNET: { setup }, | ||
| icudt68_dat: function () { throw new Error('dummy link symbol') }, | ||
| icudt73_dat: function () { throw new Error('dummy link symbol') }, |
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.
Shouldn't it be icudt72_dat?
ToDo: Investigate this on the future attempt.
No description provided.