diff options
author | Dmitry Vyukov <dvyukov@google.com> | 2019-10-22 14:44:29 +0200 |
---|---|---|
committer | Dmitry Vyukov <dvyukov@google.com> | 2019-11-06 09:09:21 +0000 |
commit | 0ea74401399c6bd66138693e7ca0cc54307d3289 (patch) | |
tree | 1d111416a74a8dbd19bb3dbc9b861d185f084cb8 /src/runtime/crash_test.go | |
parent | 61089984a89743af1d8bad569dcffe62f00a6970 (diff) | |
download | go-0ea74401399c6bd66138693e7ca0cc54307d3289.tar.gz go-0ea74401399c6bd66138693e7ca0cc54307d3289.zip |
runtime: remove stale runtime check in tests
The check is not relevant anymore.
The comment claims that go run does not rebuild packages,
but this is not true. And we use go build anyway.
We may have added the check because without caching
rebuilding everything starting from runtime for each test
takes a while. But now we have caching.
So from every side this check just adds code and pain.
Change-Id: Ifbbb643724100622e5f9db884339b67cde4ba729
Reviewed-on: https://go-review.googlesource.com/c/go/+/202450
Run-TryBot: Dmitry Vyukov <dvyukov@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/crash_test.go')
-rw-r--r-- | src/runtime/crash_test.go | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/src/runtime/crash_test.go b/src/runtime/crash_test.go index 0bee734a27..6c3127fa75 100644 --- a/src/runtime/crash_test.go +++ b/src/runtime/crash_test.go @@ -104,8 +104,6 @@ func buildTestProg(t *testing.T, binary string, flags ...string) (string, error) t.Skip("-quick") } - checkStaleRuntime(t) - testprog.Lock() defer testprog.Unlock() if testprog.dir == "" { @@ -152,34 +150,6 @@ func TestVDSO(t *testing.T) { } } -var ( - staleRuntimeOnce sync.Once // guards init of staleRuntimeErr - staleRuntimeErr error -) - -func checkStaleRuntime(t *testing.T) { - staleRuntimeOnce.Do(func() { - // 'go run' uses the installed copy of runtime.a, which may be out of date. - out, err := testenv.CleanCmdEnv(exec.Command(testenv.GoToolPath(t), "list", "-gcflags=all="+os.Getenv("GO_GCFLAGS"), "-f", "{{.Stale}}", "runtime")).CombinedOutput() - if err != nil { - staleRuntimeErr = fmt.Errorf("failed to execute 'go list': %v\n%v", err, string(out)) - return - } - if string(out) != "false\n" { - t.Logf("go list -f {{.Stale}} runtime:\n%s", out) - out, err := testenv.CleanCmdEnv(exec.Command(testenv.GoToolPath(t), "list", "-gcflags=all="+os.Getenv("GO_GCFLAGS"), "-f", "{{.StaleReason}}", "runtime")).CombinedOutput() - if err != nil { - t.Logf("go list -f {{.StaleReason}} failed: %v", err) - } - t.Logf("go list -f {{.StaleReason}} runtime:\n%s", out) - staleRuntimeErr = fmt.Errorf("Stale runtime.a. Run 'go install runtime'.") - } - }) - if staleRuntimeErr != nil { - t.Fatal(staleRuntimeErr) - } -} - func testCrashHandler(t *testing.T, cgo bool) { type crashTest struct { Cgo bool |