]> granicus.if.org Git - icu/commitdiff
ICU-21028 Modifies ICU data generator so that the CLDR version is
authorgnrunge <nrunge@google.com>
Tue, 22 Sep 2020 23:26:26 +0000 (16:26 -0700)
committerNorbert Runge <41129501+gnrunge@users.noreply.github.com>
Wed, 23 Sep 2020 02:51:09 +0000 (19:51 -0700)
no longer added to the .txt data files with the exception of the
root.txt files. This will drastically limit the number of changed
files during a version upgrade because the version change is the
only change in many of the data files.

ICU-21028 Fixed an indent.

ICU-21028 Factor in feedback from review.

tools/cldr/cldr-to-icu/src/main/java/org/unicode/icu/tool/cldrtoicu/LdmlConverter.java

index 29a4966aab443c3abc0937a0f2c593c60f9302e8..0b842dcc050e02038ed681b5a19f36949c6746d8 100644 (file)
@@ -348,7 +348,9 @@ public final class LdmlConverter {
                 });
 
                 if (!splitData.getPaths().isEmpty() || isBaseLanguage || dir.includeEmpty()) {
-                    splitData.setVersion(cldrVersion);
+                    if (id.equals("root")) {
+                        splitData.setVersion(cldrVersion);
+                    }
                     write(splitData, outDir, false);
                     writtenLocaleIds.put(dir, id);
                 }
@@ -561,7 +563,7 @@ public final class LdmlConverter {
         } else {
             // These empty files only exist because the target of an alias has a parent locale
             // which is itself not in the set of written ICU files. An "indirect alias target".
-            icuData.setVersion(config.getVersionInfo().getCldrVersion());
+            // No need to add data: Just write a resource bundle with an empty top-level table.
         }
         write(icuData, dir, false);
     }