Replace ICU build tools in CLDR project with new package in ICU project
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:
[Java package "org.unicode.icu.tool.cldrtoicu"]
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.
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/...
As I say, figure out what you want and I'll do it. I don't even have an opinion...
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/… ?
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).