diff options
author | Russ Cox <rsc@golang.org> | 2015-01-20 21:17:17 -0500 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2015-01-21 02:19:27 +0000 |
commit | db52315c88e588b87895dcb159d1b4886f355e92 (patch) | |
tree | 2098a2eae61be4f907b65659ea4da78f5355992a /src/make.rc | |
parent | 725e3a7afbb5b07e9345dd9ef6878af0cd959776 (diff) | |
download | go-db52315c88e588b87895dcb159d1b4886f355e92.tar.gz go-db52315c88e588b87895dcb159d1b4886f355e92.zip |
[dev.cc] build: do not 'clean -i std' between dist and go_bootstrap
Executing 'clean -i std' removes installed Go programs,
including the toolchain binaries we need for building.
It's not clear why the 'clean -i std' is here in the first place.
cmd/dist just removed the entire pkg tree, so everything is new.
The only reason for 'clean -i std' would be if you don't trust
that dist compiled the packages properly. If that's true for
some reason, we can fix cmd/dist, or add -a to the install
commands that follow. Perhaps clean -i std should not
remove tools, or perhaps std should not expand to any tools.
Not sure.
Also remove banner from make.bat and make.rc that was
already removed from make.bash. cmd/dist prints it now.
Also fix array size error in liblink/objfile.c.
Fixes dev.cc build.
Change-Id: I60855e001a682efce55ad9aa307a8f3ee47f7366
Reviewed-on: https://go-review.googlesource.com/3100
Reviewed-by: Russ Cox <rsc@golang.org>
Diffstat (limited to 'src/make.rc')
-rwxr-xr-x | src/make.rc | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/make.rc b/src/make.rc index 5fe30aced7..593ebc8f2f 100755 --- a/src/make.rc +++ b/src/make.rc @@ -72,14 +72,12 @@ if(~ $1 --dist-tool){ exit } -echo '# Building compilers and Go bootstrap tool for host,' $GOHOSTOS/$GOHOSTARCH^. buildall = -a if(~ $1 --no-clean) buildall = () ./cmd/dist/dist bootstrap $buildall -v # builds go_bootstrap # Delay move of dist tool to now, because bootstrap may clear tool directory. mv cmd/dist/dist $GOTOOLDIR/dist -$GOTOOLDIR/go_bootstrap clean -i std echo # Run only one process at a time on 9vx. |