code review 12149043: doc: don't invoke go build with -o /dev/null during test

doc: don't invoke go build with -o /dev/null during test

Fixes Issue 5998.

[kortschak, 2013-07-31 07:23:12]

Hello golang-dev@googlegroups.com,

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

[khr1, 2013-07-31 15:22:13]

LGTM.


On Wed, Jul 31, 2013 at 12:23 AM, <dan.kortschak@adelaide.edu.au> 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:
> doc: don't invoke go build with -o /dev/null during test
>
> Fixes Issue 5998.
>
> Please review this at
https://codereview.appspot.**com/12149043/<https://codereview.appspot.com/121...
>
> Affected files:
>   M doc/codewalk/run
>
>
> Index: doc/codewalk/run
> ==============================**==============================**=======
> --- a/doc/codewalk/run
> +++ b/doc/codewalk/run
> @@ -17,5 +17,5 @@
>  go run pig.go | grep 'Wins, losses staying at k = 100: 210/990 (21.2%),
> 780/990 (78.8%)' > /dev/null || fail pig
>
>  # sharemem.xml: only build the example, as it uses the network
> -go build -o /dev/null urlpoll.go || fail urlpoll
> -
> +go build urlpoll.go || fail urlpoll
> +rm -f urlpoll
>
>
> --
>
> ---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...
> .
>
>
>

[r, 2013-08-01 01:03:19]

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

doc: don't invoke go build with -o /dev/null during test

Fixes Issue 5998.

R=golang-dev, khr
CC=golang-dev
https://codereview.appspot.com/12149043

Committer: Rob Pike <r@golang.org>