Skip to content

Commit

Permalink
selfinstall: Fix self install in Makefile and install.sh
Browse files Browse the repository at this point in the history
We were doing `npm install -g` on `make install` and in the `install.sh`
bootstrapper, which now makes a symlink.  This updates it to go back to
creating a global copy.

Fixes: #17554
Fixes: #17377
Fixes: #16916

Credit: @iarna
Reviewed-By: @zkat
PR-URL: npm/npm#17626
  • Loading branch information
iarna committed Jul 10, 2017
1 parent fb82207 commit b58ec8e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ latest:
node bin/npm-cli.js install -g -f npm ${NPMOPTS}

install: all
node bin/npm-cli.js install -g -f ${NPMOPTS}
node bin/npm-cli.js install -g -f ${NPMOPTS} $(shell node bin/npm-cli.js pack | tail -1)

# backwards compat
dev: install
Expand Down
2 changes: 1 addition & 1 deletion scripts/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ cd "$TMP" \
fi
if [ "$make" = "NOMAKE" ]; then
"$node" bin/npm-cli.js rm npm -gf
"$node" bin/npm-cli.js install -gf
"$node" bin/npm-cli.js install -gf $("$node" bin/npm-cli.js pack | tail -1)
fi) \
&& cd "$BACK" \
&& rm -rf "$TMP" \
Expand Down

0 comments on commit b58ec8e

Please sign in to comment.