scripts: Always perform setlocal and endlocal calls in pairs

Ensure that there isn't a mismatch between setlocal and endlocal calls,
which could have happened due to setlocal being called after certain
error conditions were checked for.

diff --git a/buildconf.bat b/buildconf.bat
index b5f68d6d70d1c64cb0162c401c4e585e78e74d4e..25f1928bfaf135bdbbd7a4beea6a5758419f21c9 100644 (file)
--- a/buildconf.bat
+++ b/buildconf.bat
@@ -30,16 +30,16 @@ rem This file is not included or required for curl's release archives or daily
 rem snapshot archives.
 
 :begin
+  rem Set our variables
+  setlocal
+  set MODE=GENERATE
+
   rem Switch to this batch file's directory
   cd /d "%~0\.." 1>NUL 2>&1
 
   rem Check we are running from a curl git repository
   if not exist GIT-INFO goto norepo
 
-  rem Set our variables
-  setlocal
-  set MODE=GENERATE
-
 :parseArgs
   if "%~1" == "" goto start
 

diff --git a/projects/generate.bat b/projects/generate.bat
index 36778e810df02fe5e5161a0a3ec0e8a3fd27cc7b..01e62a53ce2f30fda617aff30c35a29f3ed6c85a 100644 (file)
--- a/projects/generate.bat
+++ b/projects/generate.bat
@@ -32,6 +32,11 @@ rem If you need to set the errorlevel do this instead: CALL :seterr [#]
   rem Check we are running on a Windows NT derived OS
   if not "%OS%" == "Windows_NT" goto nodos
 
+  rem Set our variables
+  setlocal ENABLEEXTENSIONS
+  set VERSION=ALL
+  set MODE=GENERATE
+
   rem Check we are not running on a network drive
   if "%~d0."=="\\." goto nonetdrv
 
@@ -41,11 +46,6 @@ rem If you need to set the errorlevel do this instead: CALL :seterr [#]
   rem Check we are running from a curl git repository
   if not exist ..\GIT-INFO goto norepo
 
-  rem Set our variables
-  setlocal ENABLEEXTENSIONS
-  set VERSION=ALL
-  set MODE=GENERATE
-
   rem Detect programs. HAVE_<PROGNAME>
   rem When not found the variable is set undefined. The undefined pattern
   rem allows for statements like "if not defined HAVE_PERL (command)"
@@ -515,7 +515,7 @@ rem Returns exit code 0 on success or 1 on failure.
   exit /b %EXITCODE%
 
 :error
-  endlocal
+  if "%OS%" == "Windows_NT" endlocal
   exit /B 1
 
 :success