Node: test-icu-transcode fails built without intl

Created on 8 Nov 2016  路  6Comments  路  Source: nodejs/node

  • Version: v8.0.0-pre
  • Platform: all
  • Subsystem: test


test-icu-transcode fails built without intl as below.
````
$ ./configure --with-intl=none
$ make
$ ./node test/parallel/test-icu-transcode.js
/home/ohtsu/github/node/test/parallel/test-icu-transcode.js:19
const dest = buffer.transcode(orig, 'utf8', test);
^

TypeError: buffer.transcode is not a function
at Object. (/home/ohtsu/github/node/test/parallel/test-icu-transcode.js:19:23)
at Module._compile (module.js:573:32)
at Object.Module._extensions..js (module.js:582:10)
at Module.load (module.js:490:32)
at tryModuleLoad (module.js:449:12)
at Function.Module._load (module.js:441:3)
at Module.runMain (module.js:607:10)
at run (bootstrap_node.js:420:7)
at startup (bootstrap_node.js:139:9)
at bootstrap_node.js:535:3
````
This issue is reserved to be fixed in the course of Code and Learn in Node Fest Tokyo 2016.

Ref: https://github.com/nodejs/code-and-learn/issues/58

good first issue i18n-api test

Most helpful comment

No one worked in this issue so it is open to everyone.

All 6 comments

No one worked in this issue so it is open to everyone.

It might be stupid question. buffer.transcode is added in: v7.1.0 ref. When I run test, is there any way to change version ?

@sotayamashita You can build Node and run tests on the master branch of the node repository in github which is already v8.0.0-pre .

I'd like to take a look on it.

@shigeki I got it. I try to test and understand what intl and icu is. thanks.

@gbrmachado Before creating a PR, I should know those mentioned above first so You can create it.

This issue was resolved in a0e13dae1fa64bc135784f444840a9a8fe53f001. Thanks! @watilde

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cong88 picture cong88  路  3Comments

Brekmister picture Brekmister  路  3Comments

Icemic picture Icemic  路  3Comments

stevenvachon picture stevenvachon  路  3Comments

srl295 picture srl295  路  3Comments