Commit 906f9791 authored by Russ Cox's avatar Russ Cox

cmd/dist: wait for pending tests before exiting

When 'go tool dist test' stops, it was intended that it first wait for
pending background tests, like a failed compilation waits for pending
background compiles. But these three lines prevented that.
Fix by deleting them. (The actual loop already contains the correct
logic to avoid running the others and to wait for what's left.)

Change-Id: I4e945495ada903fb0af567910626241bc1c52ba6
Reviewed-on: https://go-review.googlesource.com/18232Reviewed-by: default avatarIan Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent 4b0bc7c3
...@@ -844,9 +844,6 @@ func (t *tester) runPending(nextTest *distTest) { ...@@ -844,9 +844,6 @@ func (t *tester) runPending(nextTest *distTest) {
if w.err != nil { if w.err != nil {
log.Printf("Failed: %v", w.err) log.Printf("Failed: %v", w.err)
t.failed = true t.failed = true
if !t.keepGoing {
break
}
} }
} }
if t.failed && !t.keepGoing { if t.failed && !t.keepGoing {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment