Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken on Node.js 9.x #20

Closed
MylesBorins opened this issue Dec 4, 2017 · 3 comments
Closed

Broken on Node.js 9.x #20

MylesBorins opened this issue Dec 4, 2017 · 3 comments

Comments

@MylesBorins
Copy link

When running npm install we receive the following error

full-icu@1.2.1-0 postinstall /Users/mborins/code/full-icu-npm
node postinstall.js

npm install icu4c-data@60l (Node 9.2.0 and small-icu 60.1) -> icudt60l.dat
full-icu$ /Users/mborins/.nvm/versions/node/v9.2.0/bin/node /Users/mborins/.nvm/versions/node/v9.2.0/lib/node_modules/npm/bin/npm-cli.js install icu4c-data@60l
npm ERR! code ETARGET
npm ERR! notarget No matching version found for icu4c-data@60l
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR! A complete log of this run can be found in:
npm ERR! /Users/mborins/.npm/_logs/2017-12-04T20_48_48_472Z-debug.log
/Users/mborins/code/full-icu-npm/install-spawn.js:41
throw(Error(cmdPath + ' ' + args.join(' ') + ' --> status ' + spawned.status));

It would appear icu4c-data@60l is not published

We are going to have to skip full-icu-test in CITGM until this is resolved

MylesBorins added a commit to MylesBorins/citgm that referenced this issue Dec 4, 2017
Test suite currently does not support the version of full-icu packaged
in Node.js 9.x

refs: nodejs/full-icu-npm#20
@refack
Copy link

refack commented Dec 4, 2017

Refs: http://bugs.icu-project.org/trac/ticket/11617#comment:7

@srl295
Copy link
Member

srl295 commented Dec 5, 2017

@refack thanks… the GH issue is sufficient. Anwyay,should be fixed now, please retest

@srl295
Copy link
Member

srl295 commented Dec 5, 2017

it now works for me, reopen if not…

@srl295 srl295 closed this as completed Dec 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants