diff options
author | Todd Gamblin <tgamblin@llnl.gov> | 2016-05-05 00:53:58 -0700 |
---|---|---|
committer | Todd Gamblin <tgamblin@llnl.gov> | 2016-05-05 00:53:58 -0700 |
commit | c6a7af3fa7fc8239a1d8bbfd02d4669a095adba2 (patch) | |
tree | bcbb92002e92cd0fc68b5131cd2584b8fdb67874 | |
parent | d00680a88357434848a177536c062d2e7e904813 (diff) | |
parent | 0583bc98598d5c868e7ca402cbdf80bb4557c8cf (diff) | |
download | spack-c6a7af3fa7fc8239a1d8bbfd02d4669a095adba2.tar.gz spack-c6a7af3fa7fc8239a1d8bbfd02d4669a095adba2.tar.bz2 spack-c6a7af3fa7fc8239a1d8bbfd02d4669a095adba2.tar.xz spack-c6a7af3fa7fc8239a1d8bbfd02d4669a095adba2.zip |
Merge pull request #885 from davydden/p4est_test_option
p4est: put back +tests variant
-rw-r--r-- | var/spack/repos/builtin/packages/p4est/package.py | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/var/spack/repos/builtin/packages/p4est/package.py b/var/spack/repos/builtin/packages/p4est/package.py index 017c4e3fbf..c059632079 100644 --- a/var/spack/repos/builtin/packages/p4est/package.py +++ b/var/spack/repos/builtin/packages/p4est/package.py @@ -7,6 +7,8 @@ class P4est(Package): version('1.1', '37ba7f4410958cfb38a2140339dbf64f') + variant('tests', default=True, description='Run small tests') + # build dependencies depends_on('automake') depends_on('autoconf') @@ -33,5 +35,11 @@ class P4est(Package): configure('--prefix=%s' % prefix, *options) make() - make("check") + # Make tests optional as sometimes mpiexec can't be run with an error: + # mpiexec has detected an attempt to run as root. + # Running at root is *strongly* discouraged as any mistake (e.g., in + # defining TMPDIR) or bug can result in catastrophic damage to the OS + # file system, leaving your system in an unusable state. + if '+tests' in self.spec: + make("check") make("install") |