diff options
| author | Robert Findley <rfindley@google.com> | 2023-02-13 09:47:44 -0500 |
|---|---|---|
| committer | Robert Findley <rfindley@google.com> | 2023-02-13 16:28:07 +0000 |
| commit | 72d72f9ea8a7370c051936174c192a90a1164b8c (patch) | |
| tree | b452d2f1da0844b14df92b09f0e6c40894823dd5 | |
| parent | a9792b20f4084edb02ec81de82af6d908cffa128 (diff) | |
| download | golang-x-tools-72d72f9ea8a7370c051936174c192a90a1164b8c.tar.gz | |
go/ssa: temporarily skip the 'net' package in TestStdlib
Following CL 465836, go/ssa panics on the net package. Skip it for now
to unbreak the longtest builders.
I thought I may need to skip the reverse transitive cone of net, but
simply skipping the net package appears to fix the failing test.
For golang/go#58491
Change-Id: Id2fb898ba2867ecf70311ab8945ac6330dbffb0a
Reviewed-on: https://go-review.googlesource.com/c/tools/+/467676
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
| -rw-r--r-- | go/ssa/stdlib_test.go | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/go/ssa/stdlib_test.go b/go/ssa/stdlib_test.go index 8b9f4238d..f85f220b4 100644 --- a/go/ssa/stdlib_test.go +++ b/go/ssa/stdlib_test.go @@ -50,6 +50,18 @@ func TestStdlib(t *testing.T) { t.Fatal(err) } + // TODO(golang/go#58491): fix breakage on new generic code in net. + // + // For now, exclude the 'net' package. + netIdx := -1 + for i, pkg := range pkgs { + if pkg.PkgPath == "net" { + netIdx = i + break + } + } + pkgs = append(pkgs[:netIdx], pkgs[netIdx+1:]...) + t1 := time.Now() alloc1 := bytesAllocated() |