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

ICU Data customiser does not work with Node #9246

Closed
sxa opened this issue Feb 18, 2015 · 5 comments
Closed

ICU Data customiser does not work with Node #9246

sxa opened this issue Feb 18, 2015 · 5 comments
Labels

Comments

@sxa
Copy link
Member

sxa commented Feb 18, 2015

(This probably needs tagging with intl - FAO @srl295)

The wiki page at https://github.com/joyent/node/wiki/Intl#using-the-small-build suggests that you can use provide additional data at runtime using "a packaged data file /some/path/icudt53l.dat such as from http://apps.icu-project.org/datacustom/"

While I can use the data file that is supplied with the ICU source distribution I cannot get anything from the data customizer tool to work properly - it triggers a memory fault/core dump when I point node at it with the --icu-data-dir parameter and try to use the APIs (for example with the btest402.js mentioned in the wiki)

It's the same failure I get when I point the --icu-data-dir at an empty directory for which there's already an outstanding issue (https://github.com/srl295/node/issues/6 - fixed in upstream V8) - the name file from the customizer tool is identical to the one that comes out of the ICU source distribution (icudt54l.dat) and if I trace the execution it is definitely accessing the correct file, but it just doesn't seem to like it's contents later on:

stat("/dev/shm/dat.icuc/icudt54l.dat", {st_mode=S_IFREG|0644, st_size=28183472, ...}) = 0
open("/dev/shm/dat.icuc/icudt54l.dat", O_RDONLY) = 12
mmap(NULL, 28183472, PROT_READ, MAP_SHARED, 12, 0) = 0x7f2442914000
close(12) = 0

@srl295
Copy link
Member

srl295 commented Feb 18, 2015

I added a 'may not work' note to the wiki. Also
FYI @misterdjules

@misterdjules
Copy link

@srl295 Thanks for adding the note to the wiki!

@srl295
Copy link
Member

srl295 commented May 13, 2015

This may be also be caused by IcuBug #11672

@jasnell
Copy link
Member

jasnell commented Aug 26, 2015

@srl295 ... do we need to keep this one open?

@jasnell
Copy link
Member

jasnell commented Aug 29, 2015

Closing this here. Can reopen if necessary.

@jasnell jasnell closed this as completed Aug 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants