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

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')