]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: in TestScript/mod_outside, run 'go build' before checking cmd/addr2line staleness
authorBryan C. Mills <bcmills@google.com>
Mon, 21 Nov 2022 22:18:07 +0000 (17:18 -0500)
committerGopher Robot <gobot@golang.org>
Tue, 22 Nov 2022 15:58:18 +0000 (15:58 +0000)
If the build cache is cleaned (using 'go clean -cache' or similar), or
if a toolchain is freshly installed without warming the cache, the
build cache might not contain the dependencies needed to verify that
cmd/addr2line is not stale. In that case, the test should refill the
cache instead of failing.

Fixes #56889.

Change-Id: Ic6cf13b92bafa9c795e50eb0e4e1a9ae00ee8538
Reviewed-on: https://go-review.googlesource.com/c/go/+/452458
Auto-Submit: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
src/cmd/go/testdata/script/mod_outside.txt

index 65808244e8cb4a9a50180d526018fdca3de74e7d..ed13e59f36160bd32b32ac848aaf3808cbace579 100644 (file)
@@ -205,6 +205,7 @@ stderr 'needmod[/\\]needmod.go:10:2: no required module provides package example
 
 # 'go install' for a package in GOROOT should succeed.
 # (It must be non-stale here so that the test does not write to GOROOT).
+go build -o $devnull cmd/addr2line  # make sure any necessary dependencies are present in GOCACHE
 ! stale cmd/addr2line
 go install cmd/addr2line
 ! stderr .