From cc747d6b6f30238ed6f4ecec291ca0d223710057 Mon Sep 17 00:00:00 2001
From: "Adam J. Stewart" <ajstewart426@gmail.com>
Date: Sun, 16 Apr 2017 12:23:26 -0500
Subject: [PATCH] Remove redundant cmake dependencies (#3860)

---
 var/spack/repos/builtin/packages/cppad/package.py     | 2 --
 var/spack/repos/builtin/packages/elemental/package.py | 1 -
 var/spack/repos/builtin/packages/libsplash/package.py | 1 -
 var/spack/repos/builtin/packages/pngwriter/package.py | 1 -
 var/spack/repos/builtin/packages/symengine/package.py | 4 ----
 5 files changed, 9 deletions(-)

diff --git a/var/spack/repos/builtin/packages/cppad/package.py b/var/spack/repos/builtin/packages/cppad/package.py
index e17a070294..c13c2c5be9 100644
--- a/var/spack/repos/builtin/packages/cppad/package.py
+++ b/var/spack/repos/builtin/packages/cppad/package.py
@@ -34,8 +34,6 @@ class Cppad(CMakePackage):
     version('20170114', '565a534dc813fa1289764222cd8c11ea')
     version('develop', git='https://github.com/coin-or/CppAD.git')
 
-    depends_on('cmake', type='build')
-
     def cmake_args(self):
         # This package does not obey CMAKE_INSTALL_PREFIX
         args = [
diff --git a/var/spack/repos/builtin/packages/elemental/package.py b/var/spack/repos/builtin/packages/elemental/package.py
index 24b5bf8d15..0021c7832b 100644
--- a/var/spack/repos/builtin/packages/elemental/package.py
+++ b/var/spack/repos/builtin/packages/elemental/package.py
@@ -62,7 +62,6 @@ class Elemental(CMakePackage):
     variant('scalapack', default=False,
             description='Build with ScaLAPACK library')
 
-    depends_on('cmake', type='build')
     # Note that this forces us to use OpenBLAS until #1712 is fixed
     depends_on('blas', when='~openmp_blas ~int64_blas')
     # Hack to forward variant to openblas package
diff --git a/var/spack/repos/builtin/packages/libsplash/package.py b/var/spack/repos/builtin/packages/libsplash/package.py
index eb302e596b..61ead25177 100644
--- a/var/spack/repos/builtin/packages/libsplash/package.py
+++ b/var/spack/repos/builtin/packages/libsplash/package.py
@@ -50,7 +50,6 @@ class Libsplash(CMakePackage):
     variant('mpi', default=True,
             description='Enable parallel I/O (one-file aggregation) support')
 
-    depends_on('cmake', type='build')
     depends_on('hdf5@1.8.6:')
     depends_on('hdf5+mpi', when='+mpi')
     depends_on('mpi', when='+mpi')
diff --git a/var/spack/repos/builtin/packages/pngwriter/package.py b/var/spack/repos/builtin/packages/pngwriter/package.py
index 2950129765..ef3e28aded 100644
--- a/var/spack/repos/builtin/packages/pngwriter/package.py
+++ b/var/spack/repos/builtin/packages/pngwriter/package.py
@@ -45,7 +45,6 @@ class Pngwriter(CMakePackage):
     version('0.6.0', '0a19bc55c5f6379fea7343752fd3ffae')
     version('0.5.6', 'c13bd1fdc0e331a246e6127b5f262136')
 
-    depends_on('cmake', type='build')
     depends_on('libpng')
     depends_on('zlib')
     depends_on('freetype')
diff --git a/var/spack/repos/builtin/packages/symengine/package.py b/var/spack/repos/builtin/packages/symengine/package.py
index 136575e48c..575509cd5e 100644
--- a/var/spack/repos/builtin/packages/symengine/package.py
+++ b/var/spack/repos/builtin/packages/symengine/package.py
@@ -55,10 +55,6 @@ class Symengine(CMakePackage):
     variant('shared',       default=True,
             description='Enables the build of shared libraries')
 
-    # Build dependencies
-    depends_on('cmake',    type='build')
-
-    # Other dependencies
     # NOTE: mpir is a drop-in replacement for gmp
     # NOTE: [mpc,mpfr,flint,piranha] could also be built against mpir
     depends_on('boost',    when='+boostmp')
-- 
GitLab