code review 2592041: build: add gobuilder and goplay to run.bash

build: add gobuilder and goplay to run.bash
gobuilder: fix build to work with new log package

[adg, 2010-10-19 23:43:23]

Hello rsc (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[adg, 2010-10-19 23:51:32]

On 20 October 2010 10:43,  <adg@golang.org> wrote:
> +       ../../misc/dashboard/builder\
> +       ../../misc/goplay\

I tried initially to make these not install, but for pkg/ the default
target is install (packages need to be installed for the subsequent
packages and commands to build). They have sensible names, and go to
go/bin by default, so I'm not too opposed to having them installed if
they're already being built.

[adg, 2010-10-20 03:18:00]

Hello rsc (cc: golang-dev@googlegroups.com),

Please take another look.

[rsc1, 2010-10-20 03:21:02]

http://codereview.appspot.com/2592041/diff/5002/src/make.bash
File src/make.bash (right):

http://codereview.appspot.com/2592041/diff/5002/src/make.bash#newcode62
src/make.bash:62: for i in lib9 libbio libmach cmd pkg ../misc/dashboard/builder
../misc/goplay
make.bash is only for the things that need building
to use go.  these two would belong better in run.bash

[adg, 2010-10-20 05:41:16]

Hello rsc (cc: golang-dev@googlegroups.com),

Please take another look.

[rsc1, 2010-10-20 14:43:07]

LGTM

[adg, 2010-10-20 23:46:15]

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

build: add gobuilder and goplay to run.bash
gobuilder: fix build to work with new log package

R=rsc
CC=golang-dev
http://codereview.appspot.com/2592041