Skip to content

Commit

Permalink
Fix mpi logic in CABLE SPD (#77)
Browse files Browse the repository at this point in the history
Currently the CABLE spack build leaves the `CABLE_MPI` CMake variable
unset when `~mpi` is satisfied and relies on the default value for
`CABLE_MPI`. This change removes the if condition around
`define_from_variant` so that `CABLE_MPI` is specified to CMake
regardless of how the mpi variant is set.

Fixes #76
  • Loading branch information
SeanBryan51 authored Mar 18, 2024
1 parent 61f9d4d commit abeeaad
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions packages/cable/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,5 @@ def url_for_version(self, version):

def cmake_args(self):
args = []
if self.spec.satisfies("+mpi"):
args.append(self.define_from_variant("CABLE_MPI", "mpi"))
args.append(self.define_from_variant("CABLE_MPI", "mpi"))
return args

0 comments on commit abeeaad

Please sign in to comment.