ICU 56 step 2: Make the tree ready for local modifications

ICU 56 step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
R=mark@chromium.org

Committed: https://chromium.googlesource.com/chromium/deps/icu/+/27b09232f96145d44d3845ccc2abb94761e9b267

[jungshik at Google, 2016-01-23 07:19:43]

Description was changed from

==========
ICU update step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
==========

to

==========
ICU update step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
==========

[jungshik at Google, 2016-01-27 18:01:49]

Description was changed from

==========
ICU update step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
==========

to

==========
ICU 56 step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
==========

[jungshik at Google, 2016-01-28 08:46:32]

jshin@chromium.org changed reviewers:
+ mark@chromium.org

[jungshik at Google, 2016-01-28 08:46:32]

Hi Mark, 

Can you take a look at the changes in two scripts?

This is CL #2 in a series of 6 CLs to update ICU to 56.1. 

CL #1 (https://codereview.chromium.org/1621843002) is huge and you'd better not
open it (it can make Chrome unresponsive for a while). I'll just land it with
TBR (if you don't mind) followed by 5 other CLs in the series. It's just
updating our ICU tree to the upstream 56.2 with preserved local configuration
files using update.sh (landed a couple of weeks ago). 

This CL is followed by CL #3 : https://codereview.chromium.org/1625623003

Thank you

[Mark Mentovai, 2016-01-28 14:17:52]

LGTM

https://codereview.chromium.org/1624643003/diff/20001/android/patch_locale.sh
File android/patch_locale.sh (right):

https://codereview.chromium.org/1624643003/diff/20001/android/patch_locale.sh...
android/patch_locale.sh:10: patch -p1 < android/brkitr.patch || { echo "failed
to patch"; exit 1; }
Use stderr to your advantage: { echo "failed to patch" >2; exit 1 }

https://codereview.chromium.org/1624643003/diff/20001/scripts/trim_data.sh
File scripts/trim_data.sh (right):

https://codereview.chromium.org/1624643003/diff/20001/scripts/trim_data.sh#ne...
scripts/trim_data.sh:184: treeroot="$(dirname $0)/.."
"$(dirname "$0")/.."

[jungshik at Google, 2016-01-29 08:14:28]

Thanks ! With the review comments addressed, I'm landing it.

https://codereview.chromium.org/1624643003/diff/20001/android/patch_locale.sh
File android/patch_locale.sh (right):

https://codereview.chromium.org/1624643003/diff/20001/android/patch_locale.sh...
android/patch_locale.sh:10: patch -p1 < android/brkitr.patch || { echo "failed
to patch"; exit 1; }
On 2016/01/28 14:17:51, Mark Mentovai wrote:
> Use stderr to your advantage: { echo "failed to patch" >2; exit 1 }

You meant '>&2', didn't you? I was not thinking when adding '>2' to this CL. 
Later (in step 6 CL), I fixed it (changed it to  '>&2'. )

https://codereview.chromium.org/1624643003/diff/20001/scripts/trim_data.sh
File scripts/trim_data.sh (right):

https://codereview.chromium.org/1624643003/diff/20001/scripts/trim_data.sh#ne...
scripts/trim_data.sh:184: treeroot="$(dirname $0)/.."
On 2016/01/28 14:17:51, Mark Mentovai wrote:
> "$(dirname "$0")/.."

Done.

[jungshik at Google, 2016-01-29 08:48:01]

Description was changed from

==========
ICU 56 step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
==========

to

==========
ICU 56 step 2

Make the tree ready for the application Google's and Chrome's data
and post-56 code patches.

1. Fix trim_data.sh to run from anywhere.
2. Update patch_locale.sh for Android and add en_IN to the locale list
3. Apply data.build.patch
4. Exclude non-UI locale data for unit locale category
5. Add some regional variant locales to locale, unit, zone and coll.
6. Update locale lists for locale, unit, zone, and coll

BUG=575007
TEST=None
R=mark@chromium.org

Committed:
https://chromium.googlesource.com/chromium/deps/icu/+/27b09232f96145d44d3845c...
==========

[jungshik at Google, 2016-01-29 08:48:02]

Committed patchset #3 (id:40001) manually as
27b09232f96145d44d3845ccc2abb94761e9b267 (presubmit successful).