From 86708833c911dd01597d43f2dc4570b35f1d9aaa Mon Sep 17 00:00:00 2001 From: Keith Lazuka Date: Tue, 8 Dec 2020 12:28:24 -0500 Subject: [PATCH] [Fix] Use local variable when looping over args When the `nvm` function is called by a script which itself uses a variable named `i`, `nvm` clobbers the caller's variable. This happens even if the caller has declared its variable as local. See note 1 on https://tldp.org/LDP/abs/html/localvar.html#FTN.AEN18568 --- nvm.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/nvm.sh b/nvm.sh index 5a27ae6..2e85e6a 100644 --- a/nvm.sh +++ b/nvm.sh @@ -2554,6 +2554,7 @@ nvm() { return $? fi + local i for i in "$@" do case $i in