code review 91610046: cmd/go: fix coverage for 'package foo_test' tests

cmd/go: fix coverage for 'package foo_test' tests

Fixes issue 8062.

[rsc, 2014-05-21 16:08:34]

Hello r (cc: golang-codereviews@googlegroups.com),

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

[r, 2014-05-21 16:20:13]

LGTM

https://codereview.appspot.com/91610046/diff/40001/src/cmd/go/test.bash
File src/cmd/go/test.bash (right):

https://codereview.appspot.com/91610046/diff/40001/src/cmd/go/test.bash#newco...
src/cmd/go/test.bash:568: if grep '[^0-9]0\.0%' testdata/cover.txt >/dev/null;
then
grep -q would avoid the >

[rsc, 2014-05-21 17:58:47]

On Wed, May 21, 2014 at 12:20 PM, <r@golang.org> wrote:

> https://codereview.appspot.com/91610046/diff/40001/src/
> cmd/go/test.bash#newcode568
> src/cmd/go/test.bash:568: if grep '[^0-9]0\.0%' testdata/cover.txt
>
>> /dev/null; then
>>
> grep -q would avoid the >
>

>/dev/null is more typing but avoids depending on specific versions of
grep. maybe -q is everywhere but i can never remember (certainly not on
plan 9; there it was -s).

russ

[rsc, 2014-05-21 17:59:17]

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

cmd/go: fix coverage for 'package foo_test' tests

Fixes issue 8062.

LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/91610046