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

source command also necessary following the install script step #704

Closed
wants to merge 1 commit into from
Closed

Conversation

aj07mm
Copy link

@aj07mm aj07mm commented Mar 26, 2015

here is the Stackoverflow quesiton: http://stackoverflow.com/questions/16904658/node-version-manager-install-nvm-command-not-found/17707224#17707224

seems that this is a common problem. I installed via curl but still i had to do the source comand to get the nvm running.

@ljharb
Copy link
Member

ljharb commented Mar 26, 2015

Related to #592 - it seems to be incredibly difficult to a) properly detect which shell you're running in a portable and generic way, and b) properly detect which profile file is the appropriate one to modify.

Please use . and not source, since it's more portable, and please modify this note since the vast majority of nvm users don't need to do anything following the install script - this only applies to certain Linux distributions, or people who have modified their system shells.

@ljharb ljharb added the installing node Issues with installing node/io.js versions. label Mar 26, 2015
@ljharb ljharb added installing nvm: profile detection Issues with nvm's own install script, related to figuring out the right profile file. needs followup We need some info or action from whoever filed this issue/PR. labels Apr 6, 2015
madarche added a commit to madarche/nvm that referenced this pull request Apr 19, 2015
Using `.` and not `source`, since it's more portable

cf. nvm-sh#704 and https://wiki.ubuntu.com/DashAsBinSh#source
@madarche madarche mentioned this pull request Apr 19, 2015
nuriel added a commit to nuriel/nvm that referenced this pull request Jan 10, 2016
reflect the need to optionally source nvm after curl based installation.
nvm-sh#704 nvm-sh#748 and http://stackoverflow.com/a/17707224/690866
@ljharb ljharb closed this Jun 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
informational installing node Issues with installing node/io.js versions. installing nvm: profile detection Issues with nvm's own install script, related to figuring out the right profile file. needs followup We need some info or action from whoever filed this issue/PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants