From cfe232042981972dc0c7e8d741a04556ecaae3c3 Mon Sep 17 00:00:00 2001 From: Shenghou Ma Date: Sun, 6 Oct 2019 08:03:44 -0400 Subject: [PATCH] os: re-enable TestPipeThreads on darwin CL 197938 actually fixes those regression on Darwin as syscalls are no longer labeled as always blocking and consume a thread. Fixes #33953 Fixes #32326 Change-Id: I82c98516c23cd36f762bc5433d7b71ea8939a0ac Reviewed-on: https://go-review.googlesource.com/c/go/+/199477 Run-TryBot: Minux Ma TryBot-Result: Gobot Gobot Reviewed-by: Tobias Klauser --- src/os/os_test.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/os/os_test.go b/src/os/os_test.go index 974374ec66..b94ab0d232 100644 --- a/src/os/os_test.go +++ b/src/os/os_test.go @@ -2254,8 +2254,6 @@ func TestPipeThreads(t *testing.T) { t.Skip("skipping on Plan 9; does not support runtime poller") case "js": t.Skip("skipping on js; no support for os.Pipe") - case "darwin": - t.Skip("skipping on Darwin; issue 33953") } threads := 100 -- 2.48.1