]> Cypherpunks repositories - gostls13.git/commitdiff
make.bat: fix compare between GOROOT and srcdir paths, when either contains whitespace.
authorDominic Della Valle <ddvpublic@Gmail.com>
Sun, 13 Sep 2020 17:03:48 +0000 (17:03 +0000)
committerGiovanni Bajo <rasky@develer.com>
Sun, 13 Sep 2020 19:17:09 +0000 (19:17 +0000)
CL 96455 brings CL 57753 to Windows
However, a path comparison within it was left unquoted.
If the Go source directory resides in a path containing whitespace,
the interpreter will compare against the first portion of the path string,
and treat the remainder as an expression.
This patch amends that.

For example, consider the path
`C:\Users\Dominic Della Valle\Projects\Go\goroot\src`
Issuing `make.bat` will print out `'Della' is not recognized as an internal or external command, operable program or batch file.` before proceeding.

Change-Id: Ifcec159baeec940c29c61aa721c64c13c6fd8c14
GitHub-Last-Rev: 809ddbb4dbc80d834f8108ca44c2826016d78d1c
GitHub-Pull-Request: golang/go#41319
Reviewed-on: https://go-review.googlesource.com/c/go/+/253898
Run-TryBot: Giovanni Bajo <rasky@develer.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Giovanni Bajo <rasky@develer.com>
Trust: Giovanni Bajo <rasky@develer.com>

src/make.bat

index 277a34d5d7a0e2912e4d4144ad353df1881d00c4..b4a8e7084905e5487559779842596f35d89335e0 100644 (file)
@@ -77,7 +77,7 @@ if not "x%GOROOT_BOOTSTRAP%"=="x" goto bootstrapset
 for /f "tokens=*" %%g in ('where go 2^>nul') do (\r
        if "x%GOROOT_BOOTSTRAP%"=="x" (\r
                for /f "tokens=*" %%i in ('%%g env GOROOT 2^>nul') do (\r
-                       if /I not %%i==%GOROOT_TEMP% (\r
+                       if /I not "%%i"=="%GOROOT_TEMP%" (\r
                                set GOROOT_BOOTSTRAP=%%i\r
                        )\r
                )\r