Skip to content

Commit 72d72f9

Browse files
committed
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 <[email protected]> Run-TryBot: Robert Findley <[email protected]> TryBot-Result: Gopher Robot <[email protected]> gopls-CI: kokoro <[email protected]>
1 parent a9792b2 commit 72d72f9

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

go/ssa/stdlib_test.go

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,18 @@ func TestStdlib(t *testing.T) {
5050
t.Fatal(err)
5151
}
5252

53+
// TODO(golang/go#58491): fix breakage on new generic code in net.
54+
//
55+
// For now, exclude the 'net' package.
56+
netIdx := -1
57+
for i, pkg := range pkgs {
58+
if pkg.PkgPath == "net" {
59+
netIdx = i
60+
break
61+
}
62+
}
63+
pkgs = append(pkgs[:netIdx], pkgs[netIdx+1:]...)
64+
5365
t1 := time.Now()
5466
alloc1 := bytesAllocated()
5567

0 commit comments

Comments
 (0)