From e51b48d9a1157fa80f02881a44e35c01b4156bea Mon Sep 17 00:00:00 2001 From: dorier Date: Mon, 8 Oct 2018 08:10:03 +0100 Subject: Catalyst Package: made Python mandatory at build time (#9432) The build process of Catalyst requires Python at build time (see line 86) even when the +python variant is not selected. When the +python variant is selected, Python becomes required at build, link, and run time. There doesn't seem to be any restriction on the version of Python required when +python is not selected. --- var/spack/repos/builtin/packages/catalyst/package.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/catalyst/package.py b/var/spack/repos/builtin/packages/catalyst/package.py index c4d8dcd8b0..91f9622873 100644 --- a/var/spack/repos/builtin/packages/catalyst/package.py +++ b/var/spack/repos/builtin/packages/catalyst/package.py @@ -58,7 +58,8 @@ class Catalyst(CMakePackage): depends_on('git') depends_on('mpi') - depends_on('python@2:2.8', when='+python') + depends_on('python@2:2.8', when='+python', type=("build", "link", "run")) + depends_on('python', when='~python', type=("build")) depends_on('mesa', when='+rendering') depends_on("libx11", when='+rendering') depends_on("libxt", when='+rendering') -- cgit v1.2.3-70-g09d2