Replace ICU build tools in CLDR project with new package in ICU project

Description

The ticket contains details of this work, and everyone on the TC for both ICU and CLDR should now be aware of this task.

This ticket is for bringing in the newly rewritten transformation classes (using the new lightweight data API) into the ICU project.

New classes will live in:
/tools/cldr/cldr-to-icu/src/main/java/org/unicode/icu/tool/cldrtoicu/...
[Java package "org.unicode.icu.tool.cldrtoicu"]

Activity

Show:
David Beaumont
July 10, 2019, 8:30 PM

True. If there could reasonably be any other CLDR related code in the ICU project, then that might be a good idea.

I'm not generally a fan of 'foo2bar' naming though since it's a very English language trick to conflate "to" and "2", but in this case I'm happy to defer to the people who will be owning this code after I'm done I'll take whatever recommendation ICU team agrees on (and this all takes a couple of minutes to change even after a code drop is in).

Markus Scherer
July 10, 2019, 8:54 PM

On reflection, and looking at my own /tools/unicode/... (tools doing something with Unicode) I am less bothered by /tools/cldr/... (tools doing something with CLDR) but I am bothered by the duplication of “tools” and “cldr” in both the package path and the start of the file path.

How about /tools/cldr/src/com/ibm/icu/dev/ldml2icu/... or …/cldr2icu/… ?

David Beaumont
July 10, 2019, 9:23 PM

As I say, figure out what you want and I'll do it. I don't even have an opinion...

Markus Scherer
August 14, 2019, 6:26 PM

In today’s ICU meeting we had a long discussion of the code location. Please use /tools/cldr/cldr-to-icu/src/org/unicode/icu/tool/cldrtoicu/...

David Beaumont
March 26, 2020, 9:40 PM

I think the main body of work is done.
There's some tidyup for things like PathMatcher in the ICU code (migrating to equivalent CLDR API) but that's doable as a new ticket in 68.

Assignee

David Beaumont

Reporter

David Beaumont

Components

Labels

None

Reviewer

None

Priority

major

Time Needed

None

Fix versions

Configure