-
-
Notifications
You must be signed in to change notification settings - Fork 37.1k
deps: document which ICU version we work with #19657
Copy link
Copy link
Closed
Labels
docIssues and PRs related to Node.js documentation.Issues and PRs related to Node.js documentation.i18n-apiIssues and PRs related to Node.js internationalization support.Issues and PRs related to Node.js internationalization support.
Description
Activity
Metadata
Metadata
Assignees
Labels
docIssues and PRs related to Node.js documentation.Issues and PRs related to Node.js documentation.i18n-apiIssues and PRs related to Node.js internationalization support.Issues and PRs related to Node.js internationalization support.
in nodejs/Intl#35 I wrote:
Probably ICU4C 58.2 is the minimum for
master( ad72142 ish ) at this point. It would be 57 if #19656 were solved.Note that the backlevel ICU versions are relevant to packagers trying to use the pre-installed ICU from the system or other packaging. For example, Ubuntu stretch (at least on raspbian) has ICU 57.1 installed.
must be 99 ways to overengineer this
configurewith the relevant options?tools/icu/README.md(doesn't seem helpful)doc/api/intl.md(best option?)configureif the version is too oldconfigureif the version is too old