[Proposal] Deprecate --without-intl compilation flag

[aduh95]

Is your feature request related to a problem? Please describe.

Currently, Node.js supports the compilation with disabled ICU. I think we would benefit from officially deprecating it:

• There are already some features that are not supported, sometimes without warnings (E.G.: url: make ICU-dependent functions warn for node compiled --without-intl #35099)
• It blocks us from moving forward with the deprecation of punycode core module (punycode: runtime deprecation #35092)
• It blocks us from using Intl built-in methods in node core.
• Some of the tooling we use in core doesn't support it (E.G.: ESLint).

Describe the solution you'd like

Having a warning in the console when compiling using --without-intl flag, and document that Node.js versions compiled without ICU are expected to have some unavailable features.

Also, I think the project should allow initiatives like #27662 to land even if it doesn't work on --without-intl node versions.

If you are yourself a user of non-ICU versions of Node.js, please join the discussion, I'd be interested to hear what are the use-cases.

cc/ @nodejs/tsc

[mcollina]

+1

[targos]

Do we know if anyone builds Node with that option in the wild?

[mmomtchev]

Users of nexe, pkg and boxednode (@addaleax)

[mscdex]

I use it for reducing the binary size.

[addaleax]

Yeah, I think it’s still a decent option for decreasing binary size if the target environment profits from this. To be clear, I’m not using this myself – just saying I can see that there are valid use cases.

[targos]

What is the difference in size between --without-intl and "small ICU" ?

[aduh95]

To be clear, I'm not advocating for removal the compilation option. Maybe deprecating is not the correct wording, but I think we need to rethink the status of those builds.
I've added to the OP an example of a PR (#27662) currently blocked because of ICU which may be relevant to the discussion.

[astefanutti]

@targos

What is the difference in size between --without-intl and "small ICU" ?

You can find the size differences here:

astefanutti/scratch-node@8a588ed#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L15-L19

After I switched from --without-intl to --with-intl=small-icu in https://github.com/astefanutti/scratch-node.

[johan13]

I use it.
I run node.js on embedded devices that are updated OTA and we pay for every MB downloaded. I would like to keep --without-intl.

[wan9chi]

I use it for https://github.com/patr0nus/rust-nodejs.
It's a cargo library that embeds Node.js in Rust. It provides a no-intl feature which links Node.js compiled with --without-intl so the library consumer can get a smaller binary size.
See https://github.com/patr0nus/libnode/releases for the size differences of Node.js compiled as static libraries.

[Trott]

I think we can/should leave --without-intl as an option for people who need it, but I wonder if we could/should reduce the amount of testing we do on it. It's already basically a "might break stuff unexpectedly" option. (Maybe we should make that more clear in the doc, if it's not already sufficiently clear.)

My life would have been complete without taking time to figure out how to keep Unicode syntax highlighting from breaking the documentation build on intl. We landed something, @richardlau had to roll it back (because no one thought "this needs to be tested on full CI", which I guess is a separate issue to address), and then we had a discussion (involving @targos and @aduh95 too) about how to get it working.

I guess we could skip building docs if building --without-intl, but we may also need to skip various doc tests too.

[ChALkeR]

E.g. #35092 (review) is an example of what could have been broken with --without-intl.

[Trott]

E.g. #35092 (review) is an example of what could have been broken with --without-intl.

@ChALkeR Do you mean supporting --without-intl is holding back things like that/making them more complicated? Or do you mean that it's important to support --without-intl to prevent certain mistakes from happening?