From baa664ffaa5332d4392a9de7faad8c8f1732c932 Mon Sep 17 00:00:00 2001 From: dorier Date: Mon, 8 Oct 2018 08:08:50 +0100 Subject: Fix: bug in VTK package (#9431) * Fix: bug in VTK package Fix for issue 9423 (https://github.com/spack/spack/issues/9423): compilers other than clang don't have the `is_apple` property. * VTK package: removing trailing whitespace So flake8 stops complaining. --- var/spack/repos/builtin/packages/vtk/package.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/vtk/package.py b/var/spack/repos/builtin/packages/vtk/package.py index 8b0b3f732e..4f1d88f2ee 100644 --- a/var/spack/repos/builtin/packages/vtk/package.py +++ b/var/spack/repos/builtin/packages/vtk/package.py @@ -212,7 +212,8 @@ class Vtk(CMakePackage): # string. This fix was recommended on the VTK mailing list # in March 2014 (see # https://public.kitware.com/pipermail/vtkusers/2014-March/083368.html) - if (self.compiler.is_apple and + if (self.spec.satisfies('%clang') and + self.compiler.is_apple and self.compiler.version >= Version('5.1.0')): cmake_args.extend(['-DVTK_REQUIRED_OBJCXX_FLAGS=""']) -- cgit v1.2.3-70-g09d2