]> Cypherpunks repositories - gostls13.git/commit
cmd/dist: wait for exit in bgwait after a fatal error
authorManlio Perillo <manlio.perillo@gmail.com>
Sun, 16 May 2021 17:52:37 +0000 (19:52 +0200)
committerIan Lance Taylor <iant@golang.org>
Tue, 9 Nov 2021 21:58:03 +0000 (21:58 +0000)
commit805b4d56364ec40f29fc7efba5de537d14036c6a
treebe2e28e85a4ba706d48f5ddf1a883a45833b4dfa
parentb93220c9ca1935c56c01afc1d72d063f1e026f15
cmd/dist: wait for exit in bgwait after a fatal error

Currently, when a command scheduled by bgrun fails, bgwait returns to
the caller even in case the fatal error was detected.

In case of a syntax error in one of the standard packages, as an
example, the runInstall function will try to read the generated archive
file, only to fail since the file does not exist.  Since the runInstall
function is called in a goroutine, cmd/dist will continue to report
errors until all background goroutines are done.

Update the bgwait function to wait until program termination in case of
a fatal error, since returning to the caller (with an error, as an
example) will cause cmd/dist to report an additional error during the
next build phase.

Fixes #45410

Change-Id: If89976abad70f8d6ec79b2a5a1f2306e9c034c5a
Reviewed-on: https://go-review.googlesource.com/c/go/+/320311
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Trust: Dmitri Shuralyov <dmitshur@golang.org>
src/cmd/dist/util.go