code review 5771043: test: fix testlib to not pass an empty argument to 6g

test: fix testlib to not pass an empty argument to 6g

This change is necessary to make the run shell script work
again, but it is not sufficient as bug424.go's execution line
does not name the package that it imports.

[iant, 2012-03-07 16:51:13]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[gri, 2012-03-07 16:53:34]

LGTM

On Wed, Mar 7, 2012 at 8:51 AM,  <iant@golang.org> wrote:
> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> test: fix testlib to not pass an empty argument to 6g
>
> This change is necessary to make the run shell script work
> again, but it is not sufficient as bug424.go's execution line
> does not name the package that it imports.
>
> Please review this at http://codereview.appspot.com/5771043/
>
> Affected files:
>  M test/testlib
>
>
> Index: test/testlib
> ===================================================================
> --- a/test/testlib
> +++ b/test/testlib
> @@ -28,7 +28,7 @@
>                esac
>        done
>
> -       $G $D/$F.go "$gofiles" && $L $F.$A && ./$A.out "$@"
> +       $G $D/$F.go $gofiles && $L $F.$A && ./$A.out "$@"
>  }
>
>  cmpout() {
>
>

[rsc, 2012-03-07 16:55:15]

Oops, sorry.

[iant, 2012-03-07 19:17:00]

*** Submitted as http://code.google.com/p/go/source/detail?r=6fc5d3589066 ***

test: fix testlib to not pass an empty argument to 6g

This change is necessary to make the run shell script work
again, but it is not sufficient as bug424.go's execution line
does not name the package that it imports.

R=golang-dev, gri, rsc
CC=golang-dev
http://codereview.appspot.com/5771043