diff options
| author | Bryan C. Mills <bcmills@google.com> | 2020-05-21 14:16:50 -0400 |
|---|---|---|
| committer | Bryan C. Mills <bcmills@google.com> | 2020-05-21 21:17:48 +0000 |
| commit | 9f4aeb36e22f5c7eda76111b4c49c0434b4d2897 (patch) | |
| tree | efff143de18ada2b60dc8bd0a09c48906b380dd0 /src/text | |
| parent | 11b3730a02c93fd5745bfd977156541a9033759b (diff) | |
| download | go-9f4aeb36e22f5c7eda76111b4c49c0434b4d2897.tar.gz go-9f4aeb36e22f5c7eda76111b4c49c0434b4d2897.zip | |
all: use a hermetic "go" tool in standard-library tests
The go/build package uses the "go" tool from the user's environment,
but its tests should not assume that that tool is in any particular
state, let alone appropriate for running the test.
Instead, explicitly use testenv.GoTool, adding it to $PATH in a
TestMain when necessary.
Fixes #39199
Fixes #39198
Change-Id: I56618a55ced473e75dd96eeb3a8f7084e2e64d02
Reviewed-on: https://go-review.googlesource.com/c/go/+/234880
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Diffstat (limited to 'src/text')
| -rw-r--r-- | src/text/template/link_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/text/template/link_test.go b/src/text/template/link_test.go index b7415d29bb..4eac7e6755 100644 --- a/src/text/template/link_test.go +++ b/src/text/template/link_test.go @@ -49,7 +49,7 @@ func main() { if err := ioutil.WriteFile(filepath.Join(td, "x.go"), []byte(prog), 0644); err != nil { t.Fatal(err) } - cmd := exec.Command("go", "build", "-o", "x.exe", "x.go") + cmd := exec.Command(testenv.GoToolPath(t), "build", "-o", "x.exe", "x.go") cmd.Dir = td if out, err := cmd.CombinedOutput(); err != nil { t.Fatalf("go build: %v, %s", err, out) |