From 6251faa23d4f3998aabbdd60a4e99d34d1ee97ca Mon Sep 17 00:00:00 2001
From: Joshua Horton <jahorton@sil.org>
Date: Fri, 31 May 2024 14:52:31 +0700
Subject: [PATCH] fix(common): only retry the npm ci call itself

---
 resources/shellHelperFunctions.sh | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/resources/shellHelperFunctions.sh b/resources/shellHelperFunctions.sh
index 6bdec897d4..6d47747304 100755
--- a/resources/shellHelperFunctions.sh
+++ b/resources/shellHelperFunctions.sh
@@ -228,10 +228,6 @@ _try_multiple_times ( ) {
 # build invocation
 #
 verify_npm_setup() {
-  try_multiple_times _verify_npm_setup
-}
-
-_verify_npm_setup() {
   # We'll piggy-back on the builder module dependency build state to determine
   # if npm ci has been called in the current script invocation. Adding the
   # prefix /external/ to module name in order to differentiate between this and
@@ -249,6 +245,8 @@ _verify_npm_setup() {
     builder_die "Build environment setup error detected!  Please ensure Node.js is installed!"
 
   pushd "$KEYMAN_ROOT" > /dev/null
-  npm ci
+
+  try_multiple_times npm ci
+  
   popd > /dev/null
-}
+}
\ No newline at end of file