Skip to content

Commit 93c0992

Browse files
committed
CMake: Skip MPICXX
We do not use the MPI-2 only MPI C++ bindings, but some implemenations (esp. OpenMPI) still create an internal dependency on it, which is unnecessary and complicates tooling and deployments.
1 parent efb4f83 commit 93c0992

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/tooling.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ jobs:
6161
SOURCEPATH="$(pwd)"
6262
share/openPMD/download_samples.sh build
6363
export LDFLAGS="${LDFLAGS} -fsanitize=address,undefined -shared-libsan"
64-
export CXXFLAGS="${CXXFLAGS} -fsanitize=address,undefined -shared-libsan -DOMPI_SKIP_MPICXX"
64+
export CXXFLAGS="${CXXFLAGS} -fsanitize=address,undefined -shared-libsan"
6565
cmake -S . -B build \
6666
-DopenPMD_USE_MPI=ON \
6767
-DopenPMD_USE_PYTHON=ON \

CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,7 @@ message(STATUS "openPMD-api superbuild: ${openPMD_SUPERBUILD}")
233233
# Clang+MPI: Potentially needed MPI::MPI_C targets in the past
234234
# (exact MPI flavor & Clang version lost)
235235
# BullMPI: PUBLIC dependency to MPI::MPI_CXX is missing in MPI::MPI_C target
236+
set(MPI_CXX_SKIP_MPICXX ON) # We do not use MPI-2 only MPI C++ bindings
236237
set(openPMD_MPI_LINK_C_DEFAULT OFF)
237238
option(openPMD_MPI_LINK_C "Also link the MPI C targets" ${openPMD_MPI_LINK_C_DEFAULT})
238239
mark_as_advanced(openPMD_MPI_LINK_C)

0 commit comments

Comments
 (0)