From 1e6b5ce039295625127926283cff9d806f5181dd Mon Sep 17 00:00:00 2001
From: Matthew McEachen <matthew+github@mceachen.org>
Date: Thu, 16 Mar 2017 23:54:55 -0700
Subject: [PATCH] [Fix] stop setting mirror env vars unnecessarily

Fixes #1429
---
 nvm.sh                              | 14 ++------------
 test/fast/Unit tests/nvm_get_mirror | 19 +++++++++++++++++--
 2 files changed, 19 insertions(+), 14 deletions(-)
 mode change 100755 => 100644 nvm.sh

diff --git a/nvm.sh b/nvm.sh
old mode 100755
new mode 100644
index c3a2ff9..6e67ee2
--- a/nvm.sh
+++ b/nvm.sh
@@ -127,16 +127,6 @@ if [ -z "${NVM_DIR-}" ]; then
 fi
 unset NVM_SCRIPT_SOURCE 2> /dev/null
 
-
-# Setup mirror location if not already set
-if [ -z "${NVM_NODEJS_ORG_MIRROR-}" ]; then
-  export NVM_NODEJS_ORG_MIRROR="https://nodejs.org/dist"
-fi
-
-if [ -z "${NVM_IOJS_ORG_MIRROR-}" ]; then
-  export NVM_IOJS_ORG_MIRROR="https://iojs.org/dist"
-fi
-
 nvm_tree_contains_path() {
   local tree
   tree="${1-}"
@@ -1532,8 +1522,8 @@ nvm_is_merged_node_version() {
 
 nvm_get_mirror() {
   case "${1}-${2}" in
-    node-std) nvm_echo "${NVM_NODEJS_ORG_MIRROR}" ;;
-    iojs-std) nvm_echo "${NVM_IOJS_ORG_MIRROR}" ;;
+    node-std) nvm_echo "${NVM_NODEJS_ORG_MIRROR:-https://nodejs.org/dist}" ;;
+    iojs-std) nvm_echo "${NVM_IOJS_ORG_MIRROR:-https://iojs.org/dist}" ;;
     *)
       nvm_err 'unknown type of node.js or io.js release'
       return 1
diff --git a/test/fast/Unit tests/nvm_get_mirror b/test/fast/Unit tests/nvm_get_mirror
index 7067ff9..f4116a6 100755
--- a/test/fast/Unit tests/nvm_get_mirror	
+++ b/test/fast/Unit tests/nvm_get_mirror	
@@ -4,6 +4,9 @@ set -ex
 
 die () { echo "$@" ; exit 1; }
 
+unset NVM_NODEJS_ORG_MIRROR
+unset NVM_IOJS_ORG_MIRROR
+
 \. ../../../nvm.sh
 
 ! nvm_get_mirror || die 'unknown release type did not error'
@@ -12,5 +15,17 @@ die () { echo "$@" ; exit 1; }
 ! nvm_get_mirror node foo || die 'unknown release type did not error'
 ! nvm_get_mirror iojs foo || die 'unknown release type did not error'
 
-[ "$(nvm_get_mirror node std)" = "${NVM_NODEJS_ORG_MIRROR}" ] || die 'node std did not yield NVM_NODEJS_ORG_MIRROR'
-[ "$(nvm_get_mirror iojs std)" = "${NVM_IOJS_ORG_MIRROR}" ] || die 'node std did not yield NVM_NODEJS_ORG_MIRROR'
+[ -z "$NVM_NODEJS_ORG_MIRROR" ] || die "MIRROR environment variables should not be exported"
+[ -z "$NVM_IOJS_ORG_MIRROR" ] || die "MIRROR environment variables should not be exported"
+
+[ "$(nvm_get_mirror node std)" = "https://nodejs.org/dist" ] || die "incorrect default node-std mirror"
+[ "$(nvm_get_mirror iojs std)" = "https://iojs.org/dist" ] || die "incorrect default iojs-std mirror"
+
+NVM_NODEJS_ORG_MIRROR="test://domain"
+[ "$(nvm_get_mirror node std)" = "test://domain" ] || die "node-std mirror should respect NVM_NODEJS_ORG_MIRROR"
+unset NVM_NODEJS_ORG_MIRROR
+
+NVM_IOJS_ORG_MIRROR="test://domain"
+[ "$(nvm_get_mirror iojs std)" = "test://domain" ] || die "iojs-std mirror should respect NVM_IOJS_ORG_MIRROR"
+unset NVM_IOJS_ORG_MIRROR
+