Update install.sh #176

Merged
merged 1 commit into from Dec 4, 2012

3 participants

@witoldsz

$HOME/.nvm used instead of introduced earlier NVM_TARGET.

@witoldsz witoldsz Update install.sh
$HOME/.nvm used instead of introduced earlier NVM_TARGET.
2f6b0ca
@creationix creationix merged commit 7bee39f into creationix:master Dec 4, 2012
@koenpunt

$NVM_TARGET does not expand to the variable set in the script, so the line appears as following in your profile:

[[ -s "$NVM_TARGET/nvm.sh" ]] && . "$NVM_TARGET/nvm.sh" # This loads NVM
@witoldsz

Ah, of course, you are right.
I was thinking, maybe the NVM_TARGET should be assigned only when it is unset. That would allow users to launch install script with NVM_TARGET being preset to location of their own... or... the install location could be the parameter of the script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment