From 29ce8eb892d1cff22f5ebf28e9c0ea23246d6e5b Mon Sep 17 00:00:00 2001
From: "Kelly (KT) Thompson" <KineticTheory@users.noreply.github.com>
Date: Tue, 19 Mar 2019 08:49:30 -0600
Subject: [PATCH] Draco: add variants (#10925)

* Draco: add variants

+ This package has many optional build dependencies that were not registered in
  older versions of this recipe.  I've added (and tested) this more complete
  list of optional dependencies: parmetis, superlu-dist, qt.

* fix style issues
---
 .../repos/builtin/packages/draco/package.py   | 26 ++++++++++++-------
 1 file changed, 17 insertions(+), 9 deletions(-)

diff --git a/var/spack/repos/builtin/packages/draco/package.py b/var/spack/repos/builtin/packages/draco/package.py
index 06956ed867..bf8ebfbd6b 100644
--- a/var/spack/repos/builtin/packages/draco/package.py
+++ b/var/spack/repos/builtin/packages/draco/package.py
@@ -25,12 +25,20 @@ class Draco(CMakePackage):
     version('6_20_1', sha256='b1c51000c9557e0818014713fce70d681869c50ed9c4548dcfb2e9219c354ebe')
     version('6_20_0', sha256='a6e3142c1c90b09c4ff8057bfee974369b815122b01d1f7b57888dcb9b1128f6')
 
-    variant('lapack', default=False, description='Enable LAPACK Wrapper')
-    variant('eospac', default=False, description='Enable EOSPAC Support')
-
-    depends_on('mpi@3:')
-    depends_on('random123')
-    depends_on('gsl')
-    depends_on('python')
-    depends_on('lapack', when='+lapack')
-    depends_on('eospac', when='+eospac')
+    variant('lapack',   default=False, description='Enable LAPACK Wrapper')
+    variant('eospac',   default=False, description='Enable EOSPAC Support')
+    variant('parmetis', default=False, description='Enable Parmetis Support')
+    variant('qt',       default=False, description='Enable Qt Support')
+    variant('superlu_dist', default=False, description='Enable SuperLU-DIST Support')
+
+    depends_on('mpi@3:',    type=('build', 'run'))
+    depends_on('random123', type='build')
+    depends_on('gsl',       type='build')
+    depends_on('python',    type=('build', 'run'))
+    depends_on('numdiff',   type='run')
+
+    depends_on('lapack',   when='+lapack', type='build')
+    depends_on('eospac',   when='+eospac', type='build')
+    depends_on('parmetis', when='+parmetis', type='build')
+    depends_on('qt', when='+qt', type=('build', 'run'))
+    depends_on('superlu-dist@:5.99', when='+superlu_dist', type='build')
-- 
GitLab