v8-cmake icon indicating copy to clipboard operation
v8-cmake copied to clipboard

i18n-enabled builds

Open bnoordhuis opened this issue 5 years ago • 3 comments

Requires bundling and building Chromium's copy of ICU; see the DEPS file in the v8/ directory. We'll just have to take the repo size and build time increase on the chin.

Stretch goal: compile/link against an external copy of ICU. No guarantees as to whether it actually a) builds, and b) runs. Upstream V8 makes no such guarantees either.

bnoordhuis avatar Jan 29 '20 12:01 bnoordhuis

Maybe worth investigating using a package manager / cmake find_package for this? e.g. vcpkg and/or conan. Users can decide whether or not to pull that in if they need it then. I don't personally need this currently so would ideally want to avoid the cost it if possible.

petebannister avatar Jun 12 '23 16:06 petebannister

Chromium/V8 is built against a Google fork of ICU so I'm not 100% sure that works (or keeps working.)

Node.js vendors a different copy of ICU so it's not completely impossible but I'm a Node.js maintainer and I can confidently say it's not always smooth sailing. Upgrades are sometimes painful.

If nothing else, linking to different versions of ICU means you get different output (because of different CLDR versions) and that's just confusing. I lean toward vendoring for that reason.

bnoordhuis avatar Jun 12 '23 22:06 bnoordhuis

I see.. perhaps this is one of limited use cases where a git submodule would work well and create a new repo for the ICU fork

petebannister avatar Jun 13 '23 07:06 petebannister