Skip to content
This repository has been archived by the owner on Nov 28, 2020. It is now read-only.

Benchmarking for Node Verison 8 -broken by npm upgrade #111

Closed
mhdawson opened this issue May 27, 2017 · 5 comments
Closed

Benchmarking for Node Verison 8 -broken by npm upgrade #111

mhdawson opened this issue May 27, 2017 · 5 comments

Comments

@mhdawson
Copy link
Member

Seems like npm 5 beta in version 8 has broken the benchmark job for version 8.

Most likely because we depend on the location of the .js file as opposed to doing a full install. Need to investigate and fix up path when running with 5.

@mhdawson
Copy link
Member Author

Issue seems to be that .../node/deps/npm/cli.js does not exist in master/v8.x but did exist in earlier node versions. It was simply a wrapper that did:

#!/usr/bin/env node
require('./bin/npm-cli.js')

@mhdawson
Copy link
Member Author

Looks like instructions in Makefile changes from:

node cli.js install -g -f npm ${NPMOPTS}

to

node bin/npm-cli.js install -g -f npm ${NPMOPTS}

in: nodejs/node#12936

So looks like it is on purpose.

@mhdawson
Copy link
Member Author

Testing swapping one command for the other: https://ci.nodejs.org/view/Node.js%20benchmark/job/benchmark-footprint-master/600/console

@mhdawson
Copy link
Member Author

Looks good in the master job, just applying to the 8.x job now.

@mhdawson
Copy link
Member Author

Ok, all green again, closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant