Skip to content

Commit

Permalink
Add comment regarding specifying the compiler for MPI
Browse files Browse the repository at this point in the history
  • Loading branch information
SeanBryan51 committed Jan 28, 2024
1 parent 81a9b35 commit cfe4d7b
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions packages/cable/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,14 @@ def cmake_args(self):
if self.spec.satisfies("+mpi"):
args += [
self.define_from_variant("CABLE_MPI", "mpi"),
# TODO(Sean): we should let CMake find the MPI compiler instead
# of specifying it explicitly. However, this requires adding
# find_package(MPI) to the CMakeLists.txt file. This was not
# done in https://github.com/CABLE-LSM/CABLE/pull/200 so that
# we can demonstrate binary equivalence in the executables when
# transitioning from a Makefile based to a CMake based build
# system. Once find_package(MPI) is added, this file should be
# updated.
self.define(
"CMAKE_Fortran_COMPILER",
self.spec["mpi"].mpifc,
Expand Down

0 comments on commit cfe4d7b

Please sign in to comment.