Repository navigation
Use system icu in containers #286
Description
I know that officially distributed node binaries don't use system icu (as it's not guaranteed) but things could be seen another way with containers.
Why? especially to avoid full-icu install pain (it's been a while that it's not working well and no real solution still emerging).
- detect "full-icu" module node#3460 (opened since over 1 year)
- Fails to install full-icu yarnpkg/yarn#2124
In depth, I suggest to add the following lines when compiling node (here the alpine variant) until things get fixed into full-icu package:
Activity
chorrell commentedon Dec 7, 2016
Can the icu-libs packaged be added to .build-deps and then later removed or is it still required afterwards for icu support in node?
chorrell commentedon Dec 7, 2016
Actually, nevermind, it is required. I was confusing it with icu-dev.
So we could technically do this for the Alpine image since we're building node from source, but at some point we'd like to have (via the build group) a musl/alpine binary that we can install similarly to what we do for the other images. This change would complicate that a bit since the musl/alpine binary would presumably not be built with icu. Not having to build from source would cut down on the build time significantly and would speed up releases to the docker hub. I guess a musl/alpine binary with baked in icu support would work if we added icu-libs to the docker image.
shouze commentedon Dec 7, 2016
I guess it would work but yes icu lib is not statically linked this way. Do you think we could change that into a statically linked lib instead is it probably what
--with-intl=full-icudo when we compile node?robin-anil commentedon Apr 18, 2017
Any reason why this is stalled. We have an internationalized app and would love to get official node images with full icu support
shouze commentedon Apr 18, 2017
@r-tock ATM the best thing is to build you own containerised docker node like I've done here https://github.com/rezzza/docker-node/blob/master/vendor/karibbu/Dockerfile. BTW the real issue is more on this issue as we're using yarn and this package bring a solution to download full icu data through dependency management: nodejs/full-icu-npm#9
shouze commentedon Jun 15, 2017
Closing this one as full-icu can now be installed with npm AND yarn since from now. see nodejs/full-icu-npm#9 & last today merge nodejs/full-icu-npm#14
kachkaev commentedon Apr 18, 2018
I also struggled with incorrect number formatting inside a Docker container, but then found a solution – thanks @shouze for pointing to
full-icupackage!Here are the details of my mini-investigation: i18next/i18next#853 (comment). Hope they help those who ended up in this issue from search.
mikaelmattsson commentedon Feb 16, 2021
Switching from Node 10 to node 14 fixed it for me.
jeancabral commentedon Aug 11, 2021
for me too