Skip to content

Commit

Permalink
fix(install.sh): don't remove old npm first
Browse files Browse the repository at this point in the history
The install script will gracefully fail if things don't work.  This is
especially important for versions of npm that won't work in your current
node version.

PR-URL: #3748
Credit: @wraithgar
Close: #3748
Reviewed-by: @isaacs
  • Loading branch information
wraithgar committed Sep 13, 2021
1 parent 0320bd7 commit e4a5218
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions scripts/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -162,8 +162,6 @@ cd "$TMP" \
&& curl -qSsL -o npm.tgz "$url" \
&& $tar -xzf npm.tgz \
&& cd "$TMP"/package \
&& echo "removing existing npm" \
&& "$node" bin/npm-cli.js rm npm -gf --loglevel=silent \
&& echo "installing npm@$t" \
&& "$node" bin/npm-cli.js install -gf ../npm.tgz \
&& cd "$BACK" \
Expand Down

0 comments on commit e4a5218

Please sign in to comment.