summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTodd Gamblin <tgamblin@llnl.gov>2016-05-05 00:53:58 -0700
committerTodd Gamblin <tgamblin@llnl.gov>2016-05-05 00:53:58 -0700
commitc6a7af3fa7fc8239a1d8bbfd02d4669a095adba2 (patch)
treebcbb92002e92cd0fc68b5131cd2584b8fdb67874
parentd00680a88357434848a177536c062d2e7e904813 (diff)
parent0583bc98598d5c868e7ca402cbdf80bb4557c8cf (diff)
downloadspack-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.py10
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")