diff options
| author | Steven R. Loomis <srloomis@us.ibm.com> | 2018-10-03 23:39:14 +0300 |
|---|---|---|
| committer | Michaël Zasso <targos@protonmail.com> | 2018-10-07 13:11:34 +0300 |
| commit | 1010ef5df79c6448b5ae69c26b56f3cda8a9d826 (patch) | |
| tree | d5f2c21eef26505ee16ad5b86da3565d9c5c9d4f /tools | |
| parent | cb6818880513235cf7f6a6994ba22330a0c926a7 (diff) | |
tools: fix ICU shrinker and docs
- tools: path to ICU datafile moved
- docs: configure is now configure.py
Fixes: https://github.com/nodejs/node/issues/23245
PR-URL: https://github.com/nodejs/node/pull/23266
Reviewed-By: Refael Ackermann <refack@gmail.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/icu/README.md | 4 | ||||
| -rw-r--r-- | tools/icu/shrink-icu-src.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/tools/icu/README.md b/tools/icu/README.md index 223990bc454..01c12279311 100644 --- a/tools/icu/README.md +++ b/tools/icu/README.md @@ -92,7 +92,7 @@ make clean tools/license-builder.sh ``` -- Now, fix the default URL for the `full-icu` build in `/configure`, in +- Now, fix the default URL for the `full-icu` build in `/configure.py`, in the `configure_intl()` function. It should match the ICU URL used in the first step. When this is done, the following should build with full ICU. @@ -104,7 +104,7 @@ make make test-ci ``` -- commit the change to `configure` along with the updated `LICENSE` file. +- commit the change to `configure.py` along with the updated `LICENSE` file. - Note: To simplify review, I often will “pre-land” this patch, meaning that I run the patch through `curl -L https://github.com/nodejs/node/pull/xxx.patch diff --git a/tools/icu/shrink-icu-src.py b/tools/icu/shrink-icu-src.py index 63080603ad1..e1824354240 100644 --- a/tools/icu/shrink-icu-src.py +++ b/tools/icu/shrink-icu-src.py @@ -22,7 +22,7 @@ parser.add_option('--icu-src', parser.add_option('--icutmp', action='store', dest='icutmp', - default='out/Release/gen/icutmp', + default='out/Release/obj/gen/icutmp', help='path to icutmp dir.') |