Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

node/node-current lack ICU support #2210

Closed
vsemozhetbyt opened this issue Feb 28, 2018 · 4 comments
Closed

node/node-current lack ICU support #2210

vsemozhetbyt opened this issue Feb 28, 2018 · 4 comments

Comments

@vsemozhetbyt
Copy link
Contributor

vsemozhetbyt commented Feb 28, 2018 •
edited
Loading

Is there a reason that node/node-current are built with --without-intl flag?

This limits JavaScript and Node.js API functionality significantly: https://nodejs.org/api/intl.html#intl_options_for_building_node_js

Moreover, without small-icu support a user cannot add more ICU locales (up to full-icu npm module) at runtime: https://nodejs.org/api/intl.html#intl_providing_icu_data_at_runtime

Can node/node-current be built with --with-intl=small-icu flag?
@vsemozhetbyt
Copy link
Contributor Author

Also, one of the new RegExp features is not supported without ICU: nodejs/node#19052

@fornwall
Copy link
Member

The updated 10.10.0-1 now uses the libicu package. Let me know if anything is broken!

@vsemozhetbyt
Copy link
Contributor Author

Thank you a lot!

@vsemozhetbyt
Copy link
Contributor Author

I've checked some things from the table in https://nodejs.org/api/intl.html#intl_options_for_building_node_js and they are OK. Thank you again.

@vsemozhetbyt vsemozhetbyt mentioned this issue Sep 17, 2018
Merged
fhemberger pushed a commit to nodejs/nodejs.org that referenced this issue Sep 18, 2018
@vsemozhetbyt @fhemberger
doc: update info about Node.js installation on Android via Termux
fe6fae6 
As of
termux/termux-packages@5b08d52
`nodejs-current` package is disabled and `nodejs` package, previously
corresponded to the LTS, corresponds to the last available release.

As of
termux/termux-packages@9612920
`nodejs` package is built with `system-icu` support.
Refs: termux/termux-packages#2210
@ghost ghost locked and limited conversation to collaborators Oct 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fornwall @vsemozhetbyt