remove tmpdir on success

PREFIX not PREFIIX as per #1

README.md

@@ -35,29 +35,75 @@ wget -nv https://bit.ly/node-installer -O - ./node-installer.sh; bash ./node-ins
 
 ### Choosing a specific version
 
+**Latest**
+
 ```bash
-echo "Current node.js version is $(curl -fsSL https://nodejs.org/dist/index.tab | head -2 | tail -1 | cut -f 1)"
+export NODEJS_VER=""
+```
+
+**Exact**
+
+```bash
+export NODEJS_VER="v10.2.1"
+```
+
+**Latest of vX.Y.Z**
+
+```bash
+export NODEJS_VER="v8.1"
+```
+
+**Latest of vX.YY.Z**
+
+```bash
+# latest of v8.11.x
+export NODEJS_VER="v8.11"
+```
+
+**Latest of vX.Y**
+
+```bash
+export NODEJS_VER="v10"
+```
+
+Fun FYI, here's how the latest version is determined:
+
+```bash
+NODEJS_VER=$(curl -fsSL https://nodejs.org/dist/index.tab | tail -n +2 | cut -f 1 | head -1 )
+echo "The current node.js version is $NODEJS_VER"
 ```
 
 ```bash
-# To install a specific version rather than defaulting to latest
+BASE_VER="v10\\."
-# latest version at time of writing are v8.11.1 and v10.1.0
+NODEJS_VER=$(curl -fsSL https://nodejs.org/dist/index.tab | tail -n +2 | cut -f 1 | grep $BASE_VER | head -1 )
-export NODEJS_VER="v10.1.0"
+echo "Latest node.js $BASE_VER is $NODEJS_VER"
 ```
 
 ### Choosing an install location
 
 Just set BOTH `NODE_PATH` and `NPM_CONFIG_PREFIX`.
 The install path will be the preceding `lib/node_modules`
-(which you usually want to be the same as `NPM_CONFIG_PREFIIX` anyway).
+(which you usually want to be the same as `NPM_CONFIG_PREFIX` anyway).
 
 ```bash
-export NPM_CONFIG_PREFIX=/tmp/user/local
+export NPM_CONFIG_PREFIX=/tmp/usr/local
-export NODE_PATH=/tmp/user/local/lib/node_modules
+export NODE_PATH=/tmp/usr/local/lib/node_modules
+```
 
+A more realistic example for a self-contained node app:
+```bash
+export NPM_CONFIG_PREFIX=/opt/my-app
+export NODE_PATH=/opt/my-app/lib/node_modules
+```
+
+
+```bash
 curl -fsSL https://bit.ly/node-installer -o ./node-installer.sh; bash ./node-installer.sh --dev-deps
+```
 
-# If you want to add the install location to your PATH
+If you want to add the install location to your `PATH`:
+
+```bash
 PATH=$PATH:/tmp/user/local/bin
 ```
 
@@ -101,7 +147,7 @@ wget -nv https://bit.ly/node-installer -O /tmp/node-installer.sh; bash /tmp/node
 
 ### Other things you should know
 
-This is what gets installed:
+This is what gets installed with the dev dependencies:
 
 * rsync
 * curl

install.sh

@@ -14,7 +14,7 @@
 
 # Not every platform has or needs sudo, gotta save them O(1)s...
 sudo_cmd=""
-((EUID)) && [[ -z "$ANDROID_ROOT" ]] && sudo_cmd="sudo"
+((EUID)) && [[ -z "${ANDROID_ROOT:-}" ]] && sudo_cmd="sudo"
 
 deps_flag="$1"
 set -e
@@ -33,6 +33,8 @@ NO_FAIL2BAN="nope"
 OS="unsupported"
 ARCH=""
 SETUP_FILE=""
+my_tmp="$(mktemp -d -t node-installer.XXXXXXXX)"
+export my_tmp
 
 clear
 
@@ -110,6 +112,8 @@ elif [ "$(uname | grep -i 'Linux')" ]; then
     OS='ubuntu'
   elif [ "$(cat /etc/issue | grep -i 'elementary OS')" ]; then
     OS='ubuntu'
+  elif [ "$(cat /etc/issue | grep -i 'Arch ')" ]; then
+    OS='ubuntu'
   elif [ "$(cat /etc/issue | grep -i 'Debian')" ]; then
     OS='ubuntu'
   elif [ "$(cat /etc/issue | grep -i 'Trisquel')" ]; then
@@ -119,13 +123,15 @@ elif [ "$(uname | grep -i 'Linux')" ]; then
   elif [ "$(cat /etc/issue | grep -i 'Raspbian')" ]; then
     OS='raspbian'
   elif [ "$(cat /etc/issue | grep -i 'Fedora')" ]; then
-    OS='fedora'
+    OS='ubuntu'
   elif [ "$(cat /etc/issue | grep -i 'Marvell')" ]; then
     OS='marvell'
+  else
+    OS='ubuntu'
   fi
 else
   echo "unsupported unknown os (non-mac, non-linux)"
-  exit 1
+  exit 500
 fi
 
 case "${OS}" in
@@ -179,13 +185,13 @@ fi
 
 INSTALL_DEPS_FILE="setup-deps-${SETUP_FILE}.bash"
 INSTALL_FILE="setup-node-${SETUP_FILE}.bash"
-if [ ! -e "/tmp/${INSTALL_FILE}" ]; then
+if [ ! -e "$my_tmp/${INSTALL_FILE}" ]; then
   if [ -n "$(type -p curl)" ]; then
     curl --silent -L "${BASE_URL}/${INSTALL_FILE}" \
-      -o "/tmp/${INSTALL_FILE}" || echo 'error downloading os setup script'
+      -o "$my_tmp/${INSTALL_FILE}" || echo 'error downloading os setup script'
   elif [ -n "$(type -p wget)" ]; then
     wget --quiet "${BASE_URL}/${INSTALL_FILE}" \
-      -O "/tmp/${INSTALL_FILE}" || echo 'error downloading os setup script'
+      -O "$my_tmp/${INSTALL_FILE}" || echo 'error downloading os setup script'
   else
     echo "Found neither 'curl' nor 'wget'. Can't Continue."
     exit 1
@@ -193,13 +199,13 @@ if [ ! -e "/tmp/${INSTALL_FILE}" ]; then
 fi
 
 if [ "--dev-deps" == "$deps_flag" ]; then
-  if [ ! -e "/tmp/${INSTALL_DEPS_FILE}" ]; then
+  if [ ! -e "$my_tmp/${INSTALL_DEPS_FILE}" ]; then
     if [ -n "$(type -p curl)" ]; then
       curl --silent -L "${BASE_URL}/${INSTALL_DEPS_FILE}" \
-        -o "/tmp/${INSTALL_DEPS_FILE}" || echo 'error downloading os deps script: '"${BASE_URL}/${INSTALL_DEPS_FILE}"
+        -o "$my_tmp/${INSTALL_DEPS_FILE}" || echo 'error downloading os deps script: '"${BASE_URL}/${INSTALL_DEPS_FILE}"
     elif [ -n "$(type -p wget)" ]; then
       wget --quiet "${BASE_URL}/${INSTALL_DEPS_FILE}" \
-        -O "/tmp/${INSTALL_DEPS_FILE}" || echo 'error downloading os deps script: '"${BASE_URL}/${INSTALL_DEPS_FILE}"
+        -O "$my_tmp/${INSTALL_DEPS_FILE}" || echo 'error downloading os deps script: '"${BASE_URL}/${INSTALL_DEPS_FILE}"
     else
       echo "Found neither 'curl' nor 'wget'. Can't Continue."
       exit 1
@@ -207,14 +213,14 @@ if [ "--dev-deps" == "$deps_flag" ]; then
   fi
 fi
 
-if [ ! -e "/tmp/${INSTALL_FILE}" ]
+if [ ! -e "$my_tmp/${INSTALL_FILE}" ]
 then
   echo "Error Downloading Install File"
   exit 1
 fi
 
 if [ "--dev-deps" == "$deps_flag" ]; then
-  if [ ! -e "/tmp/${INSTALL_DEPS_FILE}" ]
+  if [ ! -e "$my_tmp/${INSTALL_DEPS_FILE}" ]
   then
     echo "Error Downloading Deps File"
     exit 1
@@ -227,10 +233,10 @@ fi
 
 if [ -z "${NODEJS_VER:-}" ]; then
   # For backwards compat
-  if [ -f "/tmp/NODEJS_VER" ]; then
+  if [ -f "$my_tmp/NODEJS_VER" ]; then
-    NODEJS_VER=$(cat /tmp/NODEJS_VER | grep v)
+    NODEJS_VER=$(cat $my_tmp/NODEJS_VER | grep v)
   elif [ -f "/tmp/IOJS_VER" ]; then
-    NODEJS_VER=$(cat /tmp/IOJS_VER | grep v)
+    NODEJS_VER=$(cat $my_tmp/IOJS_VER | grep v)
   fi
 fi
 
@@ -245,12 +251,23 @@ if [ -n "${NODEJS_VER:-}" ]; then
   fi
 fi
 
-if [ -z "${NODEJS_VER:-}" ]; then
+NODEJS_VER="${NODEJS_VER:-v}" # Search for 'v' at the least
+# sort -rV  # will sort by version number, but it appears these are already sorted
+# tail -n +2   # starts at line two (1-indexed) and all after (omits the csv header with 'version' and such)
+# cut -f 1  # gets only the first column
+# head -n 1 # gets only the most recent version
+my_char="."
+my_count=$(awk -F"${my_char}" '{print NF-1}' <<< "${NODEJS_VER}")
+# get the latest version if partial
+if [ $my_count -ne 2 ]; then
+  if [ "$NODEJS_VER" != "v" ]; then
+    NODEJS_VER="$NODEJS_VER\\."
+  fi
   if [ -n "$(type -p curl)" ]; then
-    NODEJS_VER=$(curl -fsL "$NODEJS_BASE_URL/dist/index.tab" | head -n 2 | tail -1 | cut -f 1) \
+    NODEJS_VER=$(curl -fsL "$NODEJS_BASE_URL/dist/index.tab" | tail -n +2 | cut -f 1 | grep "^$NODEJS_VER" | head -n 1) \
       || echo 'error automatically determining current node.js version'
   elif [ -n "$(type -p wget)" ]; then
-    NODEJS_VER=$(wget --quiet "$NODEJS_BASE_URL/dist/index.tab" -O - | head -n 2 | tail -1 | cut -f 1) \
+    NODEJS_VER=$(wget --quiet "$NODEJS_BASE_URL/dist/index.tab" -O - | tail -n +2 | cut -f 1 | grep "^$NODEJS_VER" | head -n 1) \
       || echo 'error automatically determining current node.js version'
   else
     echo "Found neither 'curl' nor 'wget'. Can't Continue."
@@ -278,24 +295,27 @@ if [ -e "$node_install_path/bin/node" ]; then
 # node of some version is already installed
   if [ "${NODEJS_VER}" == "$($node_install_path/bin/node -v 2>/dev/null)" ]; then
     echo node ${NODEJS_VER} is already installed
+    exit 0
   else
     echo ""
     echo "HEY, LISTEN:"
     echo ""
     echo "node.js is already installed as node $($node_install_path/bin/node -v | grep v)"
     echo ""
-    echo "to reinstall please first run: rm $node_install_path/bin/node"
+    echo "to reinstall as ${NODEJS_VER} please first run: rm $node_install_path/bin/node"
     echo ""
+    exit 1
   fi
 
   NODEJS_VER=""
 fi
 
 if [ -n "${NODEJS_VER}" ]; then
-  bash "/tmp/${INSTALL_FILE}" "${NODEJS_VER}"
+  bash "$my_tmp/${INSTALL_FILE}" "${NODEJS_VER}"
 fi
 
-$sudo_cmd chown -R $(whoami) $node_install_path/lib/node_modules
+mkdir -p $node_install_path/lib/node_modules || $sudo_cmd mkdir -p $node_install_path/lib/node_modules
+chown -R $(whoami) $node_install_path/lib/node_modules || $sudo_cmd chown -R $(whoami) $node_install_path/lib/node_modules
 
 echo ""
 
@@ -339,7 +359,7 @@ if [ "--dev-deps" == "$deps_flag" ]; then
   #  fi
   #fi
 
-  bash "/tmp/${INSTALL_DEPS_FILE}" "${NO_FAIL2BAN}"
+  bash "$my_tmp/${INSTALL_DEPS_FILE}" "${NO_FAIL2BAN}"
 
   # yarn
   #if [ -z "$(type -p yarn)" ]; then
@@ -356,3 +376,5 @@ if [ "--dev-deps" == "$deps_flag" ]; then
 
   echo ""
 fi
+
+rm -rf "$my_tmp"

setup-node-mavericks.bash

@@ -4,6 +4,9 @@ set -e
 set -u
 set -o pipefail
 
+if [ -z "${my_tmp-}" ]; then
+  my_tmp=$(mkdir -p)
+fi
 if [ -z "${PREFIX-}" ]; then
   PREFIX=""
 fi
@@ -33,8 +36,8 @@ fi
 #NODEJS_PKG="/tmp/${NODEJS_NAME}-${NODEJS_VER}.pkg"
 
 NODEJS_REMOTE="$NODEJS_BASE_URL/dist/${NODEJS_VER}/${NODEJS_NAME}-${NODEJS_VER}-darwin-x64.tar.gz"
-NODEJS_PKG="/tmp/${NODEJS_NAME}-${NODEJS_VER}-darwin-x64.tar.gz"
+NODEJS_PKG="$my_tmp/${NODEJS_NAME}-${NODEJS_VER}-darwin-x64.tar.gz"
-NODEJS_UNTAR="/tmp/${NODEJS_NAME}-${NODEJS_VER}-darwin-x64"
+NODEJS_UNTAR="$my_tmp/${NODEJS_NAME}-${NODEJS_VER}-darwin-x64"
 
 if [ -n "${NODEJS_VER}" ]; then
   echo "installing ${NODEJS_NAME} as ${NODEJS_NAME} ${NODEJS_VER}..."
@@ -47,9 +50,11 @@ if [ -n "${NODEJS_VER}" ]; then
   mkdir -p ${NODEJS_UNTAR}/
   tar xf "${NODEJS_PKG}" -C "${NODEJS_UNTAR}/" --strip-components=1
   rm -f ${NODEJS_UNTAR}/{LICENSE,CHANGELOG.md,README.md}
-  sudo rsync -a "${NODEJS_UNTAR}/" "$node_install_path/"
+
+  mkdir -p "$node_install_path/" || sudo mkdir -p "$node_install_path/"
+  rsync -a "${NODEJS_UNTAR}/" "$node_install_path/" || sudo rsync -a "${NODEJS_UNTAR}/" "$node_install_path/"
 
 
-  sudo chown -R $(whoami) "$node_install_path/lib/node_modules/"
+  chown -R $(whoami) "$node_install_path/lib/node_modules/" || sudo chown -R $(whoami) "$node_install_path/lib/node_modules/"
-  sudo chown $(whoami) "$node_install_path/bin/"
+  chown $(whoami) "$node_install_path/bin/" || sudo chown $(whoami) "$node_install_path/bin/"
 fi

setup-node-ubuntu.bash

@@ -2,12 +2,15 @@
 
 # Not every platform has or needs sudo, gotta save them O(1)s...
 sudo_cmd=""
-((EUID)) && [[ -z "$ANDROID_ROOT" ]] && sudo_cmd="sudo"
+((EUID)) && [[ -z "${ANDROID_ROOT:-}" ]] && sudo_cmd="sudo"
 
 set -e
 set -u
 set -o pipefail
 
+if [ -z "${my_tmp-}" ]; then
+  my_tmp="$(mktemp -d -t node-installer.XXXXXXXX)"
+fi
 if [ -z "${PREFIX-}" ]; then
   PREFIX=""
 fi
@@ -46,19 +49,19 @@ else
 fi
 
 NODEJS_REMOTE="${NODEJS_BASE_URL}/dist/${NODEJS_VER}/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}.tar.gz"
-NODEJS_LOCAL="/tmp/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}.tar.gz"
+NODEJS_LOCAL="$my_tmp/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}.tar.gz"
-NODEJS_UNTAR="/tmp/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}"
+NODEJS_UNTAR="$my_tmp/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}"
 
 if [ -n "${NODEJS_VER}" ]; then
   echo "installing ${NODEJS_NAME} as ${NODEJS_NAME} ${NODEJS_VER}..."
 
-  if [ -n "$(which curl 2>/dev/null)" ]; then
+  if [ -n "$(command -v curl 2>/dev/null | grep curl)" ]; then
     curl -fsSL ${NODEJS_REMOTE} -o ${NODEJS_LOCAL} || echo 'error downloading ${NODEJS_NAME}'
-  elif [ -n "$(which wget 2>/dev/null)" ]; then
+  elif [ -n "$(command -v wget 2>/dev/null | grep wget)" ]; then
     wget --quiet ${NODEJS_REMOTE} -O ${NODEJS_LOCAL} || echo 'error downloading ${NODEJS_NAME}'
   else
     echo "'wget' and 'curl' are missing. Please run the following command and try again"
-    echo "\tsudo apt-get install --yes curl wget"
+    echo "    sudo apt-get install --yes curl wget"
     exit 1
   fi
 
@@ -68,10 +71,17 @@ if [ -n "${NODEJS_VER}" ]; then
   mv ${NODEJS_UNTAR}/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH}/* ${NODEJS_UNTAR}/
   rm -rf ${NODEJS_UNTAR}/${NODEJS_NAME}-${NODEJS_VER}-linux-${ARCH} # clean up the temporary unzip folder
   rm ${NODEJS_UNTAR}/{LICENSE,CHANGELOG.md,README.md}
-  echo $sudo_cmd rsync -a "${NODEJS_UNTAR}/" "$node_install_path/"
+  if [ -n "$(command -v rsync 2>/dev/null | grep rsync)" ]; then
-  $sudo_cmd rsync -a "${NODEJS_UNTAR}/" "$node_install_path/"
+    echo $sudo_cmd rsync -Krl "${NODEJS_UNTAR}/" "$node_install_path/"
+    rsync -Krl "${NODEJS_UNTAR}/" "$node_install_path/" || $sudo_cmd rsync -Krl "${NODEJS_UNTAR}/" "$node_install_path/"
+  else
+    # due to symlink issues on Arch Linux, don't copy the share directory
+    rm -rf ${NODEJS_UNTAR}/share
+    echo $sudo_cmd cp -Hr "${NODEJS_UNTAR}/*" "$node_install_path/"
+    cp -Hr "${NODEJS_UNTAR}"/* "$node_install_path/" || $sudo_cmd cp -Hr "${NODEJS_UNTAR}"/* "$node_install_path/"
+  fi
   rm -rf "${NODEJS_UNTAR}"
 
-  $sudo_cmd chown -R $(whoami) "$node_install_path/lib/node_modules/"
+  chown -R $(whoami) "$node_install_path/lib/node_modules/" || $sudo_cmd chown -R $(whoami) "$node_install_path/lib/node_modules/"
-  $sudo_cmd chown $(whoami) ""$node_install_path"/bin/"
+  chown $(whoami) ""$node_install_path"/bin/" || $sudo_cmd chown $(whoami) ""$node_install_path"/bin/"
 fi