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_unix_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_unix_test.go')
| -rw-r--r-- | src/runtime/crash_unix_test.go | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/runtime/crash_unix_test.go b/src/runtime/crash_unix_test.go index 93cee350d0..2944c9904c 100644 --- a/src/runtime/crash_unix_test.go +++ b/src/runtime/crash_unix_test.go @@ -78,8 +78,6 @@ func TestCrashDumpsAllThreads(t *testing.T) { testenv.MustHaveGoBuild(t) - checkStaleRuntime(t) - t.Parallel() dir, err := ioutil.TempDir("", "go-build") |