Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

nodejs: Users are not aware of the "full-icu" option #46084

Closed
kevinmartin opened this issue Nov 17, 2015 · 3 comments
Closed

nodejs: Users are not aware of the "full-icu" option #46084

kevinmartin opened this issue Nov 17, 2015 · 3 comments

Comments

@kevinmartin
Copy link

Greetings from the Node.js Intl Working Group.

When installing the node, although there is an option to install with-full-icu, we feel that the option is hidden to the average user.

It would be great if a message could be displayed after installation (when installing without full-icu support) informing the user that they only have English locale support and that if they would like full support, to reinstall the formula with the with-full-icu option.

/cc @srl295 @mapmeld

@DomT4
Copy link
Member

DomT4 commented Nov 17, 2015

Don't have a massive problem with the proposed idea.

What are Node doing upstream these days? Is small-icu support the default in your binaries?

@srl295
Copy link
Contributor

srl295 commented Nov 17, 2015

@DomT4 it's the default, but we are looking into how to add a checkbox (or equivalent) for "click here to install full data to a well known spot". For brew, the full support could be a separate bottle that depends on the core node one.

@DomT4 DomT4 closed this as completed in de802bc Nov 17, 2015
@DomT4
Copy link
Member

DomT4 commented Nov 17, 2015

Added a caveat in de802bc.

If full support ever became the upstream default we'd be happy to change accordingly. Making it a separate formula is tricky as doing so would violate our broad rule that one formula should not dump in another's prefix. We've made exceptions recently, but we prefer not to when at all possible.

Let us know if we can help further at all.

@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants