d937b029a9
PR-URL: https://github.com/nodejs/node/pull/29213 Reviewed-By: Steven R Loomis <srloomis@us.ibm.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> |
||
---|---|---|
.. | ||
patches/64/source | ||
current_ver.dep | ||
icu_small.json | ||
icu_versions.json | ||
icu-generic.gyp | ||
icu-system.gyp | ||
iculslocs.cc | ||
icutrim.py | ||
no-op.cc | ||
README.md | ||
shrink-icu-src.py |
Notes about the tools/icu
subdirectory
This directory contains tools, data, and information about the ICU (International Components for Unicode) integration. ICU is used to provide internationalization functionality.
patches/
are one-off patches, actually entire source file replacements, organized by ICU version number.icu_small.json
controls the "small" (English only) ICU. It is input toicutrim.py
icu-generic.gyp
is the build file used for most ICU builds within ICU.icu-system.gyp
is an alternate build file used when--with-intl=system-icu
is invoked. It builds against thepkg-config
located ICU.iculslocs.cc
is source for theiculslocs
utility, invoked byicutrim.py
as part of repackaging. Not used separately. See source for more details.no-op.cc
— empty function to convince gyp to use a C++ compiler.README.md
— you are hereshrink-icu-src.py
— this is used during upgrade (see guide below)
How to upgrade ICU
- Make sure your Node.js workspace is clean (clean
git status
) should be sufficient. - Configure Node.js with the specific ICU version you want to upgrade to, for example:
./configure \
--with-intl=small-icu \
--with-icu-source=http://download.icu-project.org/files/icu4c/58.1/icu4c-58_1-src.tgz
make
Note
in theory, the equivalent
vcbuild.bat
commands should work also, but the commands below are makefile-centric.
-
If there are ICU version-specific changes needed, you may need to make changes in
icu-generic.gyp
or add patch files totools/icu/patches
.- Specifically, look for the lists in
sources!
in theicu-generic.gyp
for files to exclude.
- Specifically, look for the lists in
-
Verify the Node.js build works:
make test-ci
Also running
new Intl.DateTimeFormat('es', {month: 'long'}).format(new Date(9E8));
…Should return January
not enero
.
- Now, copy
deps/icu
over todeps/icu-small
python tools/icu/shrink-icu-src.py
- Now, do a clean rebuild of Node.js to test:
make -k distclean
./configure
make
- Test this newly default-generated Node.js
process.versions.icu;
new Intl.DateTimeFormat('es', {month: 'long'}).format(new Date(9E8));
(This should print your updated ICU version number, and also January
again.)
You are ready to check in the updated deps/small-icu
. This is a big commit,
so make this a separate commit from the smaller changes.
- Now, rebuild the Node.js license.
# clean up - remove deps/icu
make clean
tools/license-builder.sh
- Update the URL and hash for the full ICU file in
tools/icu/current_ver.dep
. It should match the ICU URL used in the first step. When this is done, the following should build with full ICU.
# clean up
rm -rf out deps/icu deps/icu4c*
./configure --with-intl=full-icu --download=all
make
make test-ci
-
commit the change to
tools/icu/current_ver.dep
andLICENSE
files.- 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 | git am -3 --whitespace=fix
per the collaborator’s guide… and then push that patched branch into my PR's branch. This reduces the whitespace changes that show up in the PR, since the final land will eliminate those anyway.
- Note: To simplify review, I often will “pre-land” this patch, meaning that
I run the patch through
Postscript about the tools
The files in this directory were written for the Node.js effort. It was the intent of their author (Steven R. Loomis / srl295) to merge them upstream into ICU, pending much discussion within the ICU-TC.
icu_small.json
is somewhat node-specific as it specifies a "small ICU"
configuration file for the icutrim.py
script. icutrim.py
and
iculslocs.cpp
may themselves be superseded by components built into
ICU in the future. As of this writing, however, the tools are separate
entities within Node.js, although theyare being scrutinized by interested
members of the ICU-TC. The “upstream” ICU bugs are given below.