Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ICU-21958 integrate CLDR release-42-beta2 to ICU main for 72 #2211

Conversation

pedberg-icu
Copy link
Contributor

Checklist
  • Required: Issue filed: https://unicode-org.atlassian.net/browse/ICU-21958
  • Required: The PR title must be prefixed with a JIRA Issue number.
  • Required: The PR description must include the link to the Jira Issue, for example by completing the URL in the first checklist item
  • Required: Each commit message must be prefixed with a JIRA Issue number.
  • Issue accepted (done by Technical Committee after discussion)
  • Tests included, if applicable
  • API docs and/or User Guide docs changed or added, if applicable

This integrates CLDR release-42-beta2 (the public beta for data and spec) into ICU 72. The main change that comes directly from a recent update in CLDR data is the curr/supplementalData.txt update to show that both SLE and SLL are now legal tender in Sierra Leone for a transition period.

However this PR also includes other integration cleanups that address some problems from earlier integrations as well as other changes related to the currency updates:

  1. PR ICU-22112 word break updates for @,colon; colon tailorings for fi,sv #2159 added word break updates for colon and @, plus tailorings for fi and sv. However the generation of the files brkitr/fi.txt and brkitr/sv.txt from files in data/xml/brkitr/ was not working correctly, and I have been hand-restoring those files during subsequent integrations. That is fixed here with adding missing entries in tools/cldr/cldr-to-icu/build-icu-data.xml.
  2. When there are currency code additions or deletions, or changes to legal tender status, then there is a hard-coded list used by ucurr_openISOCurrencies() that must also be updated. This PR updates that for SLE, and also for VED (whose status was updated in an earlier integration). This PR also updates corresponding tests. (This is also the subject of ICU-21989)
  3. It turns out that CLDR 41 change for sd_IN also required an update to the locales list in tools/cldr/cldr-to-icu/build-icu-data.xml, but that update was not made; consequently many stub and alias files were missing on the ICU side. This PR makes the build-icu-data.xml update and adds the corresponding stub files. (This was also the subject of ICU-21974)

@pedberg-icu
Copy link
Contributor Author

/azp run CI-Exhaustive

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@markusicu markusicu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm tnx!

@pedberg-icu pedberg-icu merged commit 49b08b4 into unicode-org:maint/maint-72 Sep 29, 2022
@pedberg-icu pedberg-icu deleted the ICU-21958-brs72-integrate-cldr-42-b2 branch September 29, 2022 17:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants