summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorTodd Gamblin <tgamblin@llnl.gov>2020-07-26 22:41:55 -0700
committerGitHub <noreply@github.com>2020-07-26 22:41:55 -0700
commitd351946194b55047226c6d653b9dc91548430006 (patch)
treecbcc773b67169db7e6cc3b505918864bc4bc18fa /lib
parent907f9e84114adab3e870f97948b604535366c68f (diff)
downloadspack-d351946194b55047226c6d653b9dc91548430006.tar.gz
spack-d351946194b55047226c6d653b9dc91548430006.tar.bz2
spack-d351946194b55047226c6d653b9dc91548430006.tar.xz
spack-d351946194b55047226c6d653b9dc91548430006.zip
bugfix: don't redundantly print ChildErrors (#17709)
A bug was introduced in #13100 where ChildErrors would be redundantly printed when raised during a build. We should eventually revisit error handling in builds and figure out what the right separation of responsibilities is for distributed builds, but for now just skip printing. - [x] SpackErrors were designed to be printed by the forked process, not by the parent, so check if they've already been printed. - [x] update tests
Diffstat (limited to 'lib')
-rw-r--r--lib/spack/spack/installer.py11
-rw-r--r--lib/spack/spack/test/cmd/install.py6
2 files changed, 12 insertions, 5 deletions
diff --git a/lib/spack/spack/installer.py b/lib/spack/spack/installer.py
index d5d15f4f99..2d4b488ac3 100644
--- a/lib/spack/spack/installer.py
+++ b/lib/spack/spack/installer.py
@@ -1568,9 +1568,14 @@ class PackageInstaller(object):
except (Exception, SystemExit) as exc:
# Best effort installs suppress the exception and mark the
# package as a failure UNLESS this is the explicit package.
- err = 'Failed to install {0} due to {1}: {2}'
- tty.error(err.format(pkg.name, exc.__class__.__name__,
- str(exc)))
+ if (not isinstance(exc, spack.error.SpackError) or
+ not exc.printed):
+ # SpackErrors can be printed by the build process or at
+ # lower levels -- skip printing if already printed.
+ # TODO: sort out this and SpackEror.print_context()
+ err = 'Failed to install {0} due to {1}: {2}'
+ tty.error(
+ err.format(pkg.name, exc.__class__.__name__, str(exc)))
self._update_failed(task, True, exc)
diff --git a/lib/spack/spack/test/cmd/install.py b/lib/spack/spack/test/cmd/install.py
index 9a46475c18..9eb2338649 100644
--- a/lib/spack/spack/test/cmd/install.py
+++ b/lib/spack/spack/test/cmd/install.py
@@ -180,10 +180,12 @@ def test_show_log_on_error(mock_packages, mock_archive, mock_fetch,
assert install.error.pkg.name == 'build-error'
assert 'Full build log:' in out
- # Message shows up for ProcessError (1), ChildError (1), and output (1)
+ print(out)
+
+ # Message shows up for ProcessError (1) and output (1)
errors = [line for line in out.split('\n')
if 'configure: error: cannot run C compiled programs' in line]
- assert len(errors) == 3
+ assert len(errors) == 2
def test_install_overwrite(