diff options
author | Steve Holme <steve_holme@hotmail.com> | 2015-08-08 11:09:08 +0100 |
---|---|---|
committer | Steve Holme <steve_holme@hotmail.com> | 2015-08-08 11:13:21 +0100 |
commit | 4a21346ef3b17f86c1878ea9df41268ae3c9ec3d (patch) | |
tree | fb7f9123f5db104efd7b33260efef239b3ba05f7 | |
parent | 4aee1f9cf5c46f56c12181f3c1bcb84466b6789c (diff) |
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.
-rw-r--r-- | buildconf.bat | 8 | ||||
-rw-r--r-- | projects/generate.bat | 12 |
2 files changed, 10 insertions, 10 deletions
diff --git a/buildconf.bat b/buildconf.bat index b5f68d6d7..25f1928bf 100644 --- 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 36778e810..01e62a53c 100644 --- 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 |