Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(535)

Delta Between Two Patch Sets: src/run.bash

Issue 7938046: code review 7938046: build, cmd/5l: actually report failures for -linkmode tests (Closed)
Left Patch Set: diff -r 9ca85035f95a https://code.google.com/p/go Created 12 years ago
Right Patch Set: diff -r 419dcca62a3d https://code.google.com/p/go/ Created 12 years ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « src/cmd/5l/obj.c ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 #!/usr/bin/env bash 1 #!/usr/bin/env bash
2 # Copyright 2009 The Go Authors. All rights reserved. 2 # Copyright 2009 The Go Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style 3 # Use of this source code is governed by a BSD-style
4 # license that can be found in the LICENSE file. 4 # license that can be found in the LICENSE file.
5 5
6 set -e 6 set -e
7 7
8 eval $(go env) 8 eval $(go env)
9 9
10 unset CDPATH # in case user has it set 10 unset CDPATH # in case user has it set
(...skipping 17 matching lines...) Expand all
28 echo '# Building packages and commands.' 28 echo '# Building packages and commands.'
29 time go install -a -v std 29 time go install -a -v std
30 echo 30 echo
31 fi 31 fi
32 32
33 # we must unset GOROOT_FINAL before tests, because runtime/debug requires 33 # we must unset GOROOT_FINAL before tests, because runtime/debug requires
34 # correct access to source code, so if we have GOROOT_FINAL in effect, 34 # correct access to source code, so if we have GOROOT_FINAL in effect,
35 # at least runtime/debug test will fail. 35 # at least runtime/debug test will fail.
36 unset GOROOT_FINAL 36 unset GOROOT_FINAL
37 37
38 # increase timeout for ARM up to 3 times the normal value
39 timeout_scale=1
40 [ "$GOARCH" == "arm" ] && timeout_scale=3
41
38 echo '# Testing packages.' 42 echo '# Testing packages.'
39 time go test std -short -timeout=120s 43 time go test std -short -timeout=$(expr 120 \* $timeout_scale)s
40 echo 44 echo
41 45
42 echo '# GOMAXPROCS=2 runtime -cpu=1,2,4' 46 echo '# GOMAXPROCS=2 runtime -cpu=1,2,4'
43 GOMAXPROCS=2 go test runtime -short -timeout=240s -cpu=1,2,4 47 GOMAXPROCS=2 go test runtime -short -timeout=$(expr 240 \* $timeout_scale)s -cpu =1,2,4
44 echo 48 echo
45 49
46 echo '# sync -cpu=10' 50 echo '# sync -cpu=10'
47 go test sync -short -timeout=120s -cpu=10 51 go test sync -short -timeout=$(expr 120 \* $timeout_scale)s -cpu=10
48 52
49 # Race detector only supported on Linux and OS X, 53 # Race detector only supported on Linux and OS X,
50 # and only on amd64, and only when cgo is enabled. 54 # and only on amd64, and only when cgo is enabled.
51 case "$GOHOSTOS-$GOOS-$GOARCH-$CGO_ENABLED" in 55 case "$GOHOSTOS-$GOOS-$GOARCH-$CGO_ENABLED" in
52 linux-linux-amd64-1 | darwin-darwin-amd64-1) 56 linux-linux-amd64-1 | darwin-darwin-amd64-1)
53 echo 57 echo
54 echo '# Testing race detector.' 58 echo '# Testing race detector.'
55 go test -race -i flag 59 go test -race -i flag
56 go test -race -short flag 60 go test -race -short flag
57 esac 61 esac
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 unset GOMAXPROCS 133 unset GOMAXPROCS
130 time go run run.go 134 time go run run.go
131 ) || exit $? 135 ) || exit $?
132 136
133 echo 137 echo
134 echo '# Checking API compatibility.' 138 echo '# Checking API compatibility.'
135 go tool api -c $GOROOT/api/go1.txt -next $GOROOT/api/next.txt -except $GOROOT/ap i/except.txt 139 go tool api -c $GOROOT/api/go1.txt -next $GOROOT/api/next.txt -except $GOROOT/ap i/except.txt
136 140
137 echo 141 echo
138 echo ALL TESTS PASSED 142 echo ALL TESTS PASSED
LEFTRIGHT

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b