code review 12873043: undo CL 12787044 / ed695cdf962b

undo CL 12787044 / ed695cdf962b

The NetBSD and OpenBSD failures are apparently real,
not due to the test bug fixed in 100b9fc0c46f.

««« original CL description
runtime/pprof: test netbsd and openbsd again

Maybe these will work now.

R=golang-dev, dvyukov, bradfitz
CC=golang-dev
https://codereview.appspot.com/12787044
»»»

[rsc, 2013-08-13 17:42:22]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go/

[bradfitz, 2013-08-13 17:43:45]

LGTM



On Tue, Aug 13, 2013 at 10:42 AM, <rsc@golang.org> wrote:

> Reviewers: golang-dev1,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go/
>
>
> Description:
> undo CL 12787044 / ed695cdf962b
>
> The NetBSD and OpenBSD failures are apparently real,
> not due to the test bug fixed in 100b9fc0c46f.
>
> ««« original CL description
> runtime/pprof: test netbsd and openbsd again
>
> Maybe these will work now.
>
> R=golang-dev, dvyukov, bradfitz
> CC=golang-dev
>
https://codereview.appspot.**com/12787044<https://codereview.appspot.com/1278...
> »»»
>
> Please review this at
https://codereview.appspot.**com/12873043/<https://codereview.appspot.com/128...
>
> Affected files:
>   M src/pkg/runtime/pprof/pprof.go
>   M src/pkg/runtime/pprof/pprof_**test.go
>
>
> Index: src/pkg/runtime/pprof/pprof.go
> ==============================**==============================**=======
> --- a/src/pkg/runtime/pprof/pprof.**go
> +++ b/src/pkg/runtime/pprof/pprof.**go
> @@ -20,7 +20,7 @@
>         "text/tabwriter"
>  )
>
> -// BUG(rsc): Profiles are incomplete and inaccuate on OS X.
> +// BUG(rsc): Profiles are incomplete and inaccuate on NetBSD, OpenBSD,
> and OS X.
>  // See http://golang.org/issue/6047 for details.
>
>  // A Profile is a collection of stack traces showing the call sequences
> Index: src/pkg/runtime/pprof/pprof_**test.go
> ==============================**==============================**=======
> --- a/src/pkg/runtime/pprof/pprof_**test.go
> +++ b/src/pkg/runtime/pprof/pprof_**test.go
> @@ -175,9 +175,9 @@
>
>  // Operating systems that are expected to fail the tests. See issue 6047.
>  var badOS = map[string]bool{
> -       "darwin": true,
> -       //"netbsd":  true,
> -       //"openbsd": true,
> +       "darwin":  true,
> +       "netbsd":  true,
> +       "openbsd": true,
>  }
>
>  func TestBlockProfile(t *testing.T) {
>
>
> --
>
> ---You received this message because you are subscribed to the Google
> Groups "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to
golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>
>
>

[dvyukov, 2013-08-13 19:33:58]

*** Submitted as https://code.google.com/p/go/source/detail?r=1563362f5697 ***

undo CL 12787044 / ed695cdf962b

The NetBSD and OpenBSD failures are apparently real,
not due to the test bug fixed in 100b9fc0c46f.

««« original CL description
runtime/pprof: test netbsd and openbsd again

Maybe these will work now.

R=golang-dev, dvyukov, bradfitz
CC=golang-dev
https://codereview.appspot.com/12787044
»»»

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/12873043

Committer: Dmitriy Vyukov <dvyukov@google.com>