clean up Kripke package and dependencies.

1 files changed, 17 insertions, 7 deletions

diff --git a/var/spack/repos/builtin/packages/kripke/package.py b/var/spack/repos/builtin/packages/kripke/package.py
index f7cc96053a..345b8af4d0 100644
--- a/var/spack/repos/builtin/packages/kripke/package.py
+++ b/var/spack/repos/builtin/packages/kripke/package.py
@@ -1,18 +1,28 @@
 from spack import *
 
 class Kripke(Package):
-    """Kripke is a simple, scalable, 3D Sn deterministic particle transport proxy/mini app."""
+    """Kripke is a simple, scalable, 3D Sn deterministic particle
+       transport proxy/mini app.
+    """
     homepage = "https://codesign.llnl.gov/kripke.php"
     url      = "https://codesign.llnl.gov/downloads/kripke-openmp-1.1.tar.gz"
 
     version('1.1', '7fe6f2b26ed983a6ce5495ab701f85bf')
 
-    #depends_on("mvapich2@1.9:")
+    variant('mpi', default=True, description='Enable MPI support')
+
+    depends_on('mpi', when="+mpi")
 
     def install(self, spec, prefix):
         with working_dir('build', create=True):
-         cmake('-DCMAKE_INSTALL_PREFIX:PATH=.', '-DENABLE_OPENMP=1', '-DENABLE_MPI=1', '..', *std_cmake_args)
-         make()
-	 #Kripke does not provide an install, so creating one here.
-	 mkdirp(prefix.bin)
-         install('kripke', prefix.bin)
+            cmake('-DCMAKE_INSTALL_PREFIX:PATH=.',
+                  '-DENABLE_OPENMP=1',
+                  '-DENABLE_MPI=1',
+                  '..',
+                  *std_cmake_args)
+            make()
+
+            # Kripke does not provide install target, so we have to copy
+            # things into place.
+            mkdirp(prefix.bin)
+            install('kripke', prefix.bin)