src/run.bash - Issue 7938046: code review 7938046: build, cmd/5l: actually report failures for -linkmode tests

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

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: 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:

Right: Side by side diff | Download

LEFT	RIGHT
(no file at all)
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 61 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
72 go run $GOROOT/test/run.go - .	72 go run $GOROOT/test/run.go - .

73 ) \|\| exit $?	73 ) \|\| exit $?

74	74

75 [ "$CGO_ENABLED" != 1 ] \|\|	75 [ "$CGO_ENABLED" != 1 ] \|\|

76 (xcd ../misc/cgo/life	76 (xcd ../misc/cgo/life

77 go run $GOROOT/test/run.go - .	77 go run $GOROOT/test/run.go - .

78 ) \|\| exit $?	78 ) \|\| exit $?

79	79

80 [ "$CGO_ENABLED" != 1 ] \|\|	80 [ "$CGO_ENABLED" != 1 ] \|\|

81 (xcd ../misc/cgo/test	81 (xcd ../misc/cgo/test

	82 set -e

82 go test -ldflags '-linkmode=auto'	83 go test -ldflags '-linkmode=auto'

83 go test -ldflags '-linkmode=internal'	84 go test -ldflags '-linkmode=internal'

84 case "$GOHOSTOS-$GOARCH" in	85 case "$GOHOSTOS-$GOARCH" in

85 darwin-386 \| darwin-amd64 \| freebsd-386 \| freebsd-amd64 \| linux-386 \| linux-amd6 4 \| netbsd-386 \| netbsd-amd64 \| openbsd-386 \| openbsd-amd64)	86 darwin-386 \| darwin-amd64 \| freebsd-386 \| freebsd-amd64 \| linux-386 \| linux-amd6 4 \| netbsd-386 \| netbsd-amd64 \| openbsd-386 \| openbsd-amd64)

86 go test -ldflags '-linkmode=external'	87 go test -ldflags '-linkmode=external'

87 esac	88 esac

88 ) \|\| exit $?	89 ) \|\| exit $?

89	90

90 [ "$CGO_ENABLED" != 1 ] \|\|	91 [ "$CGO_ENABLED" != 1 ] \|\|

91 [ "$GOHOSTOS" == windows ] \|\|	92 [ "$GOHOSTOS" == windows ] \|\|

(...skipping 40 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
132 unset GOMAXPROCS	133 unset GOMAXPROCS

133 time go run run.go	134 time go run run.go

134 ) \|\| exit $?	135 ) \|\| exit $?

135	136

136 echo	137 echo

137 echo '# Checking API compatibility.'	138 echo '# Checking API compatibility.'

138 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

139	140

140 echo	141 echo

141 echo ALL TESTS PASSED	142 echo ALL TESTS PASSED

LEFT	RIGHT