Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(195)

Unified Diff: android/patch_locale.sh

Issue 1639543006: ICU 56 step 6: Check in the pre-built ICU data (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu.git@56local_patches
Patch Set: minor clean up of README.chromium Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: android/patch_locale.sh
diff --git a/android/patch_locale.sh b/android/patch_locale.sh
index aa575fb492b5987dfcd9e3840ba5b97a32bf8a9b..929fa3d47b60caa4918f28f5c18e9734a510e701 100755
--- a/android/patch_locale.sh
+++ b/android/patch_locale.sh
@@ -86,12 +86,23 @@ done
# Excludes region data. On Android Java API is used to get the data.
+# Due to a bug in ICU, an empty region list always uses 70kB pool.res bundle.
+# As a work around, include the minimal version of en.txt
echo Overwriting region/reslocal.mk...
cat >region/reslocal.mk <<END
REGION_CLDR_VERSION = %version%
REGION_SYNTHETIC_ALIAS =
REGION_ALIAS_SOURCE =
-REGION_SOURCE =
+REGION_SOURCE = en.txt
+END
+
+echo Overwriting region/en.txt...
+cat >region/en.txt <<END
+en{
+ Countries{
+ US{"United States"}
+ }
+}
END
# On Android Java API is used to get lang data, except for the language and
@@ -179,6 +190,7 @@ for i in locales/*.txt; do
echo Overwriting $i...
sed -r '/^ calendar\{$/,/^ \}$/ {
/^ calendar\{$/p
+ /^ default\{".*"\}$/p
/^ '${CAL_PATTERN}'\{$/, /^ \}$/p
/^ \}$/p
d

Powered by Google App Engine
This is Rietveld 408576698