]> Cypherpunks repositories - gostls13.git/commitdiff
runtime/pprof: expect tests to pass on macOS
authorRuss Cox <rsc@golang.org>
Mon, 15 Feb 2021 22:02:30 +0000 (17:02 -0500)
committerRuss Cox <rsc@golang.org>
Fri, 19 Feb 2021 00:41:28 +0000 (00:41 +0000)
macOS tests have been disabled since CL 12429045 (Aug 2013).
At the time, macOS required a kernel patch to get a working profiler
(https://research.swtch.com/macpprof), which we didn't want
to require, of course.

macOS has improved - it no longer requires the kernel patch - but
we never updated the list of exceptions.

As far as I can tell, the builders have no problem passing the pprof test now.
(It is possible that the iOS builders have trouble, but that is now a different GOOS.)

Remove the exception for macOS. The test should now pass.

Fixes #6047.

Change-Id: Iab49036cacc1025e56f515bd19d084390c2f5357
Reviewed-on: https://go-review.googlesource.com/c/go/+/292229
Trust: Russ Cox <rsc@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
src/runtime/pprof/pprof_test.go

index f7c1349bc68ab19c738eb4377cf14f070f168639..d7571953a9bf45298fe9653ca8d338e503aeac68 100644 (file)
@@ -279,7 +279,7 @@ func testCPUProfile(t *testing.T, matches matchFunc, need []string, avoid []stri
 
        broken := false
        switch runtime.GOOS {
-       case "darwin", "ios", "dragonfly", "netbsd", "illumos", "solaris":
+       case "ios", "dragonfly", "netbsd", "illumos", "solaris":
                broken = true
        case "openbsd":
                if runtime.GOARCH == "arm" || runtime.GOARCH == "arm64" {