Add a flag to fail if the tests fail to build. Off by default.

Add a flag to fail if the tests fail to build. Off by default.

[sosa, 2010-03-11 00:06:37]

http://codereview.chromium.org/804006/diff/1/2
File src/scripts/build_autotest.sh (right):

http://codereview.chromium.org/804006/diff/1/2#newcode76
src/scripts/build_autotest.sh:76: [ $FLAGS_buildcheck -eq "$FLAGS_TRUE" ] &&
USE="${USE} buildcheck"
Could you either change the format of 69-74 or your format so they are
consistent.

[petkov, 2010-03-11 00:18:33]

PTAL

http://codereview.chromium.org/804006/diff/1/2
File src/scripts/build_autotest.sh (right):

http://codereview.chromium.org/804006/diff/1/2#newcode76
src/scripts/build_autotest.sh:76: [ $FLAGS_buildcheck -eq "$FLAGS_TRUE" ] &&
USE="${USE} buildcheck"
On 2010/03/11 00:06:37, sosa wrote:
> Could you either change the format of 69-74 or your format so they are
> consistent.

Done.

[sosa, 2010-03-11 00:21:03]

lgtm with nit.  Can you remove the echo?  I believe the emerge echoes the USE
anyway so we don't need to dbl echo the use

http://codereview.chromium.org/804006/diff/5001/6001
File src/scripts/build_autotest.sh (right):

http://codereview.chromium.org/804006/diff/5001/6001#newcode73
src/scripts/build_autotest.sh:73: echo USE: ${USE}
Nit:  remove debug

[petkov, 2010-03-11 00:27:59]

Submitting...

http://codereview.chromium.org/804006/diff/5001/6001
File src/scripts/build_autotest.sh (right):

http://codereview.chromium.org/804006/diff/5001/6001#newcode73
src/scripts/build_autotest.sh:73: echo USE: ${USE}
On 2010/03/11 00:21:03, sosa wrote:
> Nit:  remove debug

I didn't see them but I didn't look carefully. Remove echo anyway.