Skip to content
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

Fix LikelySubtags data for ICU73 and before #199

Open
sven-oly opened this issue Mar 13, 2024 · 0 comments
Open

Fix LikelySubtags data for ICU73 and before #199

sven-oly opened this issue Mar 13, 2024 · 0 comments

Comments

@sven-oly
Copy link
Collaborator

sven-oly commented Mar 13, 2024

LikelySubtags.txt has changed location and format in ICU74 compared to previous versions.

TODO: update date generation to consider the ICU version.

Before ICU74, the data has only the minimized and maximized values in this form:
aa{"aa_Latn_ET"}
aai{"aai_Latn_ZZ"}
aak{"aak_Latn_ZZ"}
...

ICU73 likelySubtags.txt

Note: the data is in the same place for ICU68 - ICU73 (at least)

In ICU4J 74, the test data has moved and is in two places, with identical contents.
ICU73 likelySubtags.txt

Same info is here:
icu4c/source/test/testdata/cldr/localeIdentifiers/likelySubtags.txt

// Source ; AddLikely ; RemoveFavorScript ; RemoveFavorRegion
// Source: a locale to which the following operations are applied.
// AddLikely: the result of the Add Likely Subtags.
// If Add Likely Subtags fails, then “FAIL”.
// RemoveFavorScript: Remove Likely Subtags, when the script is favored.
// Only included when different than AddLikely.
// RemoveFavorRegion: Remove Likely Subtags, when the region is favored.
// Only included when different than RemoveFavorScript.
//
// Generation: GenerateLikelyTestData.java

af ; af-Latn-ZA ; af ;
af-AQ ; af-Latn-AQ ; af-AQ ;
af-Egyp ; af-Egyp-ZA ; af-Egyp ;
af-Latn ; af-Latn-ZA ; af ;
af-NA ; af-Latn-NA ; af-NA ;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants