v0.8.0

Suppress output of `nvm use`

.travis.yml

@@ -4,5 +4,11 @@ install:
 before_script:
   - curl -o /tmp/urchin https://raw.githubusercontent.com/scraperwiki/urchin/master/urchin && chmod +x /tmp/urchin
 script:
-  - NVM_DIR=$TRAVIS_BUILD_DIR make URCHIN=/tmp/urchin test
-
+  - NVM_DIR=$TRAVIS_BUILD_DIR make TEST_SUITE=$TEST_SUITE URCHIN=/tmp/urchin $SHELL
+env:
+  - SHELL=sh TEST_SUITE=fast
+  - SHELL=dash TEST_SUITE=fast
+  - SHELL=bash TEST_SUITE=fast
+  - SHELL=zsh TEST_SUITE=fast
+  - SHELL=ksh TEST_SUITE=fast
+  - SHELL=sh TEST_SUITE=slow

CONTRIBUTING.md

@@ -5,7 +5,7 @@ We love pull requests, they're our favorite.
 However, before submitting, please review the following:
 
  - Please include tests. Changes with tests will be merged very quickly.
- - Please manually confirm that your changes, and all tests, pass in `bash`, `sh`, and `zsh`. We hope to soon automatically run all these tests, but in the meantime, we have to rely on each contributor doing so.
+ - Please manually confirm that your changes work in `bash`, `sh`/`dash`, `ksh`, and `zsh`. Fast tests do run in these shells, but it's nice to manually verify also.
  - Please maintain consistent whitespace - 2-space indentation, trailing newlines in all files, etc.
  - Any time you make a change to your PR, please rebase freshly on top of master. Nobody likes merge commits.
 

Makefile

@@ -1,15 +1,14 @@
 URCHIN=`which urchin`
 SHELLS=sh bash dash ksh zsh
+TEST_SUITE=fast
 
 .PHONY: $(SHELLS) test
 
-fast: $(SHELLS)
-
 $(SHELLS):
 	@printf '\n\033[0;34m%s\033[0m\n' "Running tests in $@"
-	@$@ $(URCHIN) -f test/fast
+	@$@ $(URCHIN) -f test/$(TEST_SUITE)
 
-test: fast
+test: $(SHELLS)
 	@$(URCHIN) -f test/slow 
 
 default: test

README.markdown

@@ -8,11 +8,11 @@ First you'll need to make sure your system has a c++ compiler.  For OSX, XCode w
 
 To install you could use the [install script][2] using cURL:
 
-    curl https://raw.githubusercontent.com/creationix/nvm/v0.7.0/install.sh | sh
+    curl https://raw.githubusercontent.com/creationix/nvm/v0.8.0/install.sh | sh
 
 or Wget:
 
-    wget -qO- https://raw.githubusercontent.com/creationix/nvm/v0.7.0/install.sh | sh
+    wget -qO- https://raw.githubusercontent.com/creationix/nvm/v0.8.0/install.sh | sh
 
 <sub>The script clones the nvm repository to `~/.nvm` and adds the source line to your profile (`~/.bash_profile`, `~/.zshrc` or `~/.profile`).</sub>
 
@@ -163,7 +163,7 @@ After the v0.8.6 release of node, nvm tries to install from binary packages. But
     nvm install -s 0.8.6
 
 [1]: https://github.com/creationix/nvm.git
-[2]: https://github.com/creationix/nvm/blob/v0.7.0/install.sh
+[2]: https://github.com/creationix/nvm/blob/v0.8.0/install.sh
 [3]: https://travis-ci.org/creationix/nvm
 [Urchin]: https://github.com/scraperwiki/urchin
 

install.sh

@@ -101,7 +101,7 @@ if [ -z "$PROFILE" ]; then
   fi
 fi
 
-SOURCE_STR="[ -s \"$NVM_DIR/nvm.sh\" ] && . \"$NVM_DIR/nvm.sh\"  # This loads nvm"
+SOURCE_STR="\nexport NVM_DIR=\"$NVM_DIR\"\n[ -s \"\$NVM_DIR/nvm.sh\" ] && . \"\$NVM_DIR/nvm.sh\"  # This loads nvm"
 
 if [ -z "$PROFILE" ] || [ ! -f "$PROFILE" ] ; then
   if [ -z $PROFILE ]; then
@@ -112,15 +112,13 @@ if [ -z "$PROFILE" ] || [ ! -f "$PROFILE" ] ; then
     echo "=> Create it (touch $PROFILE) and run this script again"
   fi
   echo "   OR"
-  echo "=> Append the following line to the correct file yourself:"
-  echo
-  echo "   $SOURCE_STR"
+  echo "=> Append the following lines to the correct file yourself:"
+  echo -e "$SOURCE_STR"
   echo
 else
   if ! grep -qc 'nvm.sh' $PROFILE; then
     echo "=> Appending source string to $PROFILE"
-    echo "" >> "$PROFILE"
-    echo $SOURCE_STR >> "$PROFILE"
+    echo -e "$SOURCE_STR" >> "$PROFILE"
   else
     echo "=> Source string already in $PROFILE"
   fi

nvm-exec

@@ -5,9 +5,9 @@ DIR="$(command cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 source "$DIR/nvm.sh"
 
 if [ -n "$NODE_VERSION" ]; then
-  nvm use $NODE_VERSION || (echo "NODE_VERSION not set" && exit 127)
+  nvm use $NODE_VERSION > /dev/null || (echo "NODE_VERSION not set" >&2 && exit 127)
 else
-  nvm use || (echo "No .nvmrc file found" && exit 127)
+  nvm use > /dev/null || (echo "No .nvmrc file found" >&2 && exit 127)
 fi
 
 exec $@

nvm.sh

@@ -130,14 +130,16 @@ nvm_ls() {
   fi
   # If it looks like an explicit version, don't do anything funny
   if [ `expr "$PATTERN" : "v[[:digit:]]*\.[[:digit:]]*\.[[:digit:]]*$"` != 0 ]; then
-    VERSIONS="$PATTERN"
+    if [ -d "$NVM_DIR/$PATTERN" ]; then
+      VERSIONS="$PATTERN"
+    fi
   else
     VERSIONS=`find "$NVM_DIR/" -maxdepth 1 -type d -name "$(nvm_format_version $PATTERN)*" -exec basename '{}' ';' \
-      | sort -t. -u -k 1.2,1n -k 2,2n -k 3,3n`
+      | sort -t. -u -k 1.2,1n -k 2,2n -k 3,3n | grep -v '^ *\.'`
   fi
   if [ -z "$VERSIONS" ]; then
-      echo "N/A"
-      return
+    echo "N/A"
+    return
   fi
   echo "$VERSIONS"
   return
@@ -179,7 +181,7 @@ nvm_checksum() {
     echo 'Checksums empty' #missing in raspberry pi binary
     return
   else
-    echo 'Checksums do not match.'
+    echo 'Checksums do not match.' >&2
     return 1
   fi
 }
@@ -249,6 +251,7 @@ nvm() {
       echo "    nvm alias <name> <version>  Set an alias named <name> pointing to <version>"
       echo "    nvm unalias <name>          Deletes the alias named <name>"
       echo "    nvm copy-packages <version> Install global NPM packages contained in <version> to current version"
+      echo "    nvm unload                  Unload NVM from shell"
       echo
       echo "Example:"
       echo "    nvm install v0.10.24        Install a specific version number"
@@ -305,7 +308,7 @@ nvm() {
         fi
         provided_version="$NVM_RC_VERSION"
       fi
-      [ -d "$NVM_DIR/$provided_version" ] && echo "$provided_version is already installed." && return
+      [ -d "$NVM_DIR/$provided_version" ] && echo "$provided_version is already installed." >&2 && return
 
       VERSION=`nvm_remote_version $provided_version`
       ADDITIONAL_PARAMETERS=''
@@ -319,13 +322,13 @@ nvm() {
       done
 
       if [ -d "$NVM_DIR/$VERSION" ]; then
-        echo "$VERSION is already installed."
+        echo "$VERSION is already installed." >&2
         nvm use "$VERSION"
         return $?
       fi
 
       if [ "$VERSION" = "N/A" ]; then
-        echo "Version '$provided_version' not found - try \`nvm ls-remote\` to browse available versions."
+        echo "Version '$provided_version' not found - try \`nvm ls-remote\` to browse available versions." >&2
         return 3
       fi
 
@@ -392,10 +395,10 @@ nvm() {
         if ! nvm_has "npm" ; then
           echo "Installing npm..."
           if [ "`expr "$VERSION" : '\(^v0\.1\.\)'`" != '' ]; then
-            echo "npm requires node v0.2.3 or higher"
+            echo "npm requires node v0.2.3 or higher" >&2
           elif [ "`expr "$VERSION" : '\(^v0\.2\.\)'`" != '' ]; then
             if [ "`expr "$VERSION" : '\(^v0\.2\.[0-2]$\)'`" != '' ]; then
-              echo "npm requires node v0.2.3 or higher"
+              echo "npm requires node v0.2.3 or higher" >&2
             else
               curl https://npmjs.org/install.sh | clean=yes npm_install=0.2.19 sh
             fi
@@ -404,7 +407,7 @@ nvm() {
           fi
         fi
       else
-        echo "nvm: install $VERSION failed!"
+        echo "nvm: install $VERSION failed!" >&2
         return 1
       fi
     ;;
@@ -412,12 +415,12 @@ nvm() {
       [ $# -ne 2 ] && nvm help && return
       PATTERN=`nvm_format_version $2`
       if [ "$PATTERN" = `nvm_version` ]; then
-        echo "nvm: Cannot uninstall currently-active node version, $PATTERN."
+        echo "nvm: Cannot uninstall currently-active node version, $PATTERN." >&2
         return 1
       fi
       VERSION=`nvm_version $PATTERN`
       if [ ! -d $NVM_DIR/$VERSION ]; then
-        echo "$VERSION version is not installed..."
+        echo "$VERSION version is not installed..." >&2
         return;
       fi
 
@@ -444,19 +447,19 @@ nvm() {
         hash -r
         echo "$NVM_DIR/*/bin removed from \$PATH"
       else
-        echo "Could not find $NVM_DIR/*/bin in \$PATH"
+        echo "Could not find $NVM_DIR/*/bin in \$PATH" >&2
       fi
       if [ `expr "$MANPATH" : ".*$NVM_DIR/.*/share/man.*"` != 0 ] ; then
         export MANPATH=`nvm_strip_path "$MANPATH" "/share/man"`
         echo "$NVM_DIR/*/share/man removed from \$MANPATH"
       else
-        echo "Could not find $NVM_DIR/*/share/man in \$MANPATH"
+        echo "Could not find $NVM_DIR/*/share/man in \$MANPATH" >&2
       fi
       if [ `expr "$NODE_PATH" : ".*$NVM_DIR/.*/lib/node_modules.*"` != 0 ] ; then
         export NODE_PATH=`nvm_strip_path "$NODE_PATH" "/lib/node_modules"`
         echo "$NVM_DIR/*/lib/node_modules removed from \$NODE_PATH"
       else
-        echo "Could not find $NVM_DIR/*/lib/node_modules in \$NODE_PATH"
+        echo "Could not find $NVM_DIR/*/lib/node_modules in \$NODE_PATH" >&2
       fi
     ;;
     "use" )
@@ -467,7 +470,7 @@ nvm() {
       if [ $# -eq 1 ]; then
         nvm_rc_version
         if [ -n "$NVM_RC_VERSION" ]; then
-            VERSION=`nvm_version $NVM_RC_VERSION`
+          VERSION=`nvm_version $NVM_RC_VERSION`
         fi
       else
         VERSION=`nvm_version $2`
@@ -480,7 +483,7 @@ nvm() {
         VERSION=`nvm_version $2`
       fi
       if [ ! -d "$NVM_DIR/$VERSION" ]; then
-        echo "$VERSION version is not installed yet"
+        echo "$VERSION version is not installed yet" >&2
         return 1
       fi
       # Strip other version from PATH
@@ -539,7 +542,7 @@ nvm() {
       fi
 
       if [ ! -d "$NVM_DIR/$VERSION" ]; then
-        echo "$VERSION version is not installed yet"
+        echo "$VERSION version is not installed yet" >&2
         return;
       fi
       RUN_NODE_PATH=`nvm_strip_path "$NODE_PATH" "/lib/node_modules"`
@@ -555,8 +558,8 @@ nvm() {
       return
     ;;
     "ls-remote" | "list-remote" )
-        nvm_print_versions "`nvm_ls_remote $2`"
-        return
+      nvm_print_versions "`nvm_ls_remote $2`"
+      return
     ;;
     "current" )
       nvm_version current
@@ -570,18 +573,18 @@ nvm() {
             DEST=`cat $ALIAS`
             VERSION=`nvm_version $DEST`
             if [ "$DEST" = "$VERSION" ]; then
-                echo "$(basename $ALIAS) -> $DEST"
+              echo "$(basename $ALIAS) -> $DEST"
             else
-                echo "$(basename $ALIAS) -> $DEST (-> $VERSION)"
+              echo "$(basename $ALIAS) -> $DEST (-> $VERSION)"
             fi
           fi
         done
         return
       fi
       if [ -z "$3" ]; then
-          rm -f $NVM_DIR/alias/$2
-          echo "$2 -> *poof*"
-          return
+        rm -f $NVM_DIR/alias/$2
+        echo "$2 -> *poof*"
+        return
       fi
       mkdir -p $NVM_DIR/alias
       VERSION=`nvm_version $3`
@@ -590,7 +593,7 @@ nvm() {
       fi
       echo $3 > "$NVM_DIR/alias/$2"
       if [ ! "$3" = "$VERSION" ]; then
-          echo "$2 -> $3 (-> $VERSION)"
+        echo "$2 -> $3 (-> $VERSION)"
       else
         echo "$2 -> $3"
       fi
@@ -598,34 +601,38 @@ nvm() {
     "unalias" )
       mkdir -p $NVM_DIR/alias
       [ $# -ne 2 ] && nvm help && return 127
-      [ ! -f "$NVM_DIR/alias/$2" ] && echo "Alias $2 doesn't exist!" && return
+      [ ! -f "$NVM_DIR/alias/$2" ] && echo "Alias $2 doesn't exist!" >&2 && return
       rm -f $NVM_DIR/alias/$2
       echo "Deleted alias $2"
     ;;
     "copy-packages" )
-        if [ $# -ne 2 ]; then
-          nvm help
-          return 127
-        fi
-        VERSION=`nvm_version $2`
-        local ROOT=`(nvm use $VERSION && npm -g root)`
-        local ROOTDEPTH=$((`echo $ROOT | sed 's/[^\/]//g'|wc -m` -1))
+      if [ $# -ne 2 ]; then
+        nvm help
+        return 127
+      fi
+      VERSION=`nvm_version $2`
+      local ROOT=`(nvm use $VERSION && npm -g root)`
+      local ROOTDEPTH=$((`echo $ROOT | sed 's/[^\/]//g'|wc -m` -1))
 
-        # declare local INSTALLS first, otherwise it doesn't work in zsh
-        local INSTALLS
-        INSTALLS=`nvm use $VERSION > /dev/null && npm -g -p ll | \grep "$ROOT\/[^/]\+$" | cut -d '/' -f $(($ROOTDEPTH + 2)) | cut -d ":" -f 2 | \grep -v npm | tr "\n" " "`
+      # declare local INSTALLS first, otherwise it doesn't work in zsh
+      local INSTALLS
+      INSTALLS=`nvm use $VERSION > /dev/null && npm -g -p ll | \grep "$ROOT\/[^/]\+$" | cut -d '/' -f $(($ROOTDEPTH + 2)) | cut -d ":" -f 2 | \grep -v npm | tr "\n" " "`
 
-        npm install -g ${INSTALLS[@]}
+      npm install -g ${INSTALLS[@]}
     ;;
     "clear-cache" )
-        rm -f $NVM_DIR/v* 2>/dev/null
-        echo "Cache cleared."
+      rm -f $NVM_DIR/v* 2>/dev/null
+      echo "Cache cleared."
     ;;
     "version" )
-        nvm_version $2
+      nvm_version $2
     ;;
     "--version" )
-        echo "0.7.0"
+      echo "0.8.0"
+    ;;
+    "unload" )
+      unset -f nvm nvm_print_versions nvm_checksum nvm_ls_remote nvm_ls nvm_remote_version nvm_version nvm_rc_version > /dev/null 2>&1
+      unset RC_VERSION NVM_NODEJS_ORG_MIRROR NVM_DIR NVM_CD_FLAGS > /dev/null 2>&1
     ;;
     * )
       nvm help

package.json

@@ -1,6 +1,6 @@
 {
   "name": "nvm",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Node Version Manager - Simple bash script to manage multiple active node.js versions",
   "directories": {
     "test": "test"

test/fast/Listing versions/Running "nvm ls" should filter out ".nvm"

@@ -0,0 +1,9 @@
+#!/bin/sh
+
+mkdir ../../../v0.1.3
+mkdir ../../../v0.2.3
+
+. ../../../nvm.sh
+
+[ -z `nvm ls | grep '^ *\.'` ]
+# The result should contain only the appropriate version numbers.

test/fast/Listing versions/Running "nvm ls" with node-like versioning vx.x.x should only list a matched version

@@ -0,0 +1,10 @@
+#!/bin/sh
+
+mkdir ../../../v0.1.2
+
+. ../../../nvm.sh
+
+nvm ls v0.1 | grep v0.1.2 &&
+nvm ls v0.1.2 | grep v0.1.2 &&
+nvm ls v0.1. | grep N/A &&
+nvm ls v0.1.1 | grep N/A

test/fast/Running "nvm install" with an invalid version fails nicely

@@ -3,5 +3,5 @@
 die () { echo $@ ; exit 1; }
 
 . ../../nvm.sh
-[ "$(nvm install invalid.invalid)" = "Version 'invalid.invalid' not found - try \`nvm ls-remote\` to browse available versions." ] || die "nvm installing an invalid version did not print a nice error message"
+[ "$(nvm install invalid.invalid 2>&1)" = "Version 'invalid.invalid' not found - try \`nvm ls-remote\` to browse available versions." ] || die "nvm installing an invalid version did not print a nice error message"
 

test/fast/Running "nvm unload" should unset all function and variables.

@@ -0,0 +1,12 @@
+#!/bin/sh
+
+fail () { echo $@ ; exit 1; }
+
+. ../../nvm.sh
+
+type nvm > /dev/null 2>&1 || fail "NVM not loaded"
+
+nvm unload
+
+type nvm > /dev/null 2>&1 && fail "NVM not unloaded" || exit 0
+

test/fast/Unit tests/nvm_prepend_path

@@ -8,11 +8,11 @@ TEST_PATH=/usr/bin:/usr/local/bin
 
 NEW_PATH=`nvm_prepend_path "$TEST_PATH" "$NVM_DIR/v0.2.5/bin"`
 
-[ "$NEW_PATH" = "$NVM_DIR/v0.2.5/bin:/usr/bin:/usr/local/bin" ] || die "Not correctly prepended: $STRIPPED_PATH "
+[ "$NEW_PATH" = "$NVM_DIR/v0.2.5/bin:/usr/bin:/usr/local/bin" ] || die "Not correctly prepended: $NEW_PATH "
 
 
 EMPTY_PATH=
 
 NEW_PATH=`nvm_prepend_path "$EMPTY_PATH" "$NVM_DIR/v0.2.5/bin"`
 
-[ "$NEW_PATH" = "$NVM_DIR/v0.2.5/bin" ] || die "Not correctly prepended: $STRIPPED_PATH "
+[ "$NEW_PATH" = "$NVM_DIR/v0.2.5/bin" ] || die "Not correctly prepended: $NEW_PATH "

test/slow/nvm install/install already installed uses it

@@ -4,7 +4,7 @@ die () { echo $@ ; exit 1; }
 
 . ../../../nvm.sh
 
-[ "$(nvm install invalid.invalid)" = "Version 'invalid.invalid' not found - try \`nvm ls-remote\` to browse available versions." ] || die "nvm installing an invalid version did not print a nice error message"
+[ "$(nvm install invalid.invalid 2>&1)" = "Version 'invalid.invalid' not found - try \`nvm ls-remote\` to browse available versions." ] || die "nvm installing an invalid version did not print a nice error message"
 
 # Remove the stuff we're clobbering.
 [ -e ../../../v0.9.7 ] && rm -R ../../../v0.9.7