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
[v12.x] icu: broken date locale string format on zh-CN #35390
Comments
Have you installed the full-icu module? You may need to reinstall it so that it installs the full ICU data to match the version of ICU used in Node.js. cc @nodejs/i18n-api |
@richardlau v12.x is built with small-icu by default (doc). Is this issue possibly related to full-icu? |
I wouldn't expect the Node.js 14 behaviour without full ICU. I can't remember if Node.js will ignore a full ICU install for the wrong version of ICU or would error/silently ignore it. |
Yes, Node.js 14 behavior is expected to working with full-icu. The issue is not if the |
the symbol |
repro'ed. Sounds like the data slicing had a failure. Some data is present, but not other data. |
cc @nodejs/i18n-api Any idea what we should do with this one? |
@richardlau sorry, i have not looked into this any further |
Closing this issue because v12.x goes EOL at the end of this month and no more releases are planned. |
What steps will reproduce the bug?
How often does it reproduce? Is there a required condition?
Always.
What is the expected behavior?
node v12.16.3 (small-icu, English only)
node v14.12.0 (full-icu)
Both cases are correct.
What do you see instead?
Invalid and meaningless symbol characters "├F10: PM┤" in localized strings.
Additional information
Reverting #33337 (5c0232a, 2d76ae7) can fix the issue.
The text was updated successfully, but these errors were encountered: