Skip to content

Commit

Permalink
Don't try to suppress use of MPI C++ binding
Browse files Browse the repository at this point in the history
This doesn't really work because we don't currently
propagate the necessary compile flags, and CMake tries
to link against the mpi_cxx library anyway.
Instead, we'll set the necessary config globally in our
build system to build RHEL packages that also work
on Fedora 40 or later (where the C++ bindings are gone).

This reverts commit 7bfed8c.
This reverts commit 841a81d.
  • Loading branch information
benmwebb committed May 6, 2024
1 parent b3a5ae8 commit 4bb1ff6
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions modules/mpi/dependency/MPI.cmake
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
# We don't need the deprecated C++ bindings, so try to suppress linkage
# against them
set(MPI_CXX_SKIP_MPICXX TRUE CACHE BOOL "" FORCE)

find_package(MPI)

if("${MPI_CXX_FOUND}")
Expand Down

0 comments on commit 4bb1ff6

Please sign in to comment.