]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: fix test failures with -count=2
authorBryan C. Mills <bcmills@google.com>
Thu, 12 Mar 2020 13:29:14 +0000 (09:29 -0400)
committerBryan C. Mills <bcmills@google.com>
Thu, 12 Mar 2020 17:46:03 +0000 (17:46 +0000)
In each test, either set the -n flag to avoid writing build artifacts
to the cache, or set GOCACHE explicitly to point to a clean cache.

Tested manually with 'go test -count=2 cmd/go'.

Fixes #37820

Change-Id: I24403e738b1a10d5fe9dc8d98ef27a76ebe2704a
Reviewed-on: https://go-review.googlesource.com/c/go/+/223140
Run-TryBot: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Jay Conrod <jayconrod@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/go/testdata/script/build_gcflags.txt
src/cmd/go/testdata/script/cgo_flag_contains_space.txt
src/cmd/go/testdata/script/gcflags_patterns.txt
src/cmd/go/testdata/script/mod_get_commit.txt
src/cmd/go/testdata/script/mod_get_tags.txt

index e0accb10d501413763495e58c0344785947b61a8..b47237410d1956ff8357f0486935f21c8ed125f4 100644 (file)
@@ -7,7 +7,8 @@ env GO111MODULE=off
 [!linux] skip  # test only works if c-archive implies -shared
 [short] skip
 
-go build -x -buildmode=c-archive -gcflags=all=-shared=false ./override.go
+env GOCACHE=$WORK/gocache  # Looking for compile commands, so need a clean cache.
+go build -x -n -buildmode=c-archive -gcflags=all=-shared=false ./override.go
 stderr '^.*/compile (.* )?-shared (.* )?-shared=false'
 
 -- override.go --
index 940340e98d4f797bdc8653c31beb2243dff572b2..a3372bbbc7672622704d58252d0751c0b30cf554 100644 (file)
@@ -1,15 +1,16 @@
 [short] skip
 [!cgo] skip
 
-go run -x main.go
+env GOCACHE=$WORK/gocache  # Looking for compile flags, so need a clean cache.
+go build -x -n main.go
 stderr '"-I[^"]+c flags"' # find quoted c flags
-! stderr '"-I[^"]+c flags".*"-I[^"]+c flags"' # don't find too many quoted c flags
+! stderr '"-I[^"]+c flags".*"-I[^"]+c flags"' # don't find too many quoted c flags per line
 stderr '"-L[^"]+ld flags"' # find quoted ld flags
-! stderr '"-L[^"]+c flags".*"-L[^"]+c flags"' # don't find too many quoted ld flags
+! stderr '"-L[^"]+c flags".*"-L[^"]+c flags"' # don't find too many quoted ld flags per line
 
 -- main.go --
 package main
 // #cgo CFLAGS: -I"c flags"
 // #cgo LDFLAGS: -L"ld flags"
 import "C"
-func main() {}
\ No newline at end of file
+func main() {}
index dce8e39715d81e2fbfad6ee782ec673fda5594e1..5374493a439e0a0c491a6ea617090d3f47d16217 100644 (file)
@@ -3,6 +3,8 @@ env GO111MODULE=off
 [!gc] skip 'using -gcflags and -ldflags'
 [short] skip
 
+env GOCACHE=$WORK/gocache  # Looking for compile commands, so need a clean cache.
+
 # -gcflags=-e applies to named packages, not dependencies
 go build -n -v -gcflags=-e z1 z2
 stderr 'compile.* -e.* -p z1'
index a906babbd5f67b8b47db53046039363125f87203..d108242c70e227b30b4d187dc4f7606ec820398a 100644 (file)
@@ -14,6 +14,9 @@ go get -d golang.org/x/text@14c0d48
 
 # dropping -d, we should see a build.
 [short] skip
+
+env GOCACHE=$WORK/gocache  # Looking for compile commands, so need a clean cache.
+
 go get -x golang.org/x/text/language@14c0d48
 stderr 'compile|cp|gccgo .*language\.a$'
 
index 603c76983f7692de5eef9eb7dc3239cd594b785b..e9869e3f0230b3ab6f6bdbb1f756071a2c624a1c 100644 (file)
@@ -14,7 +14,8 @@ stdout 'rsc.io/quote v1.5.2'
 [short] skip
 
 # Packages that are only imported in excluded files should not be built.
-go get -x .
+env GOCACHE=$WORK/gocache  # Looking for compile commands, so need a clean cache.
+go get -n -x .
 stderr 'compile.* -p m '
 ! stderr 'compile.* -p example.com/version '
 ! stderr 'compile.* -p rsc.io/quote '