Skip to content
Snippets Groups Projects
Commit f7336f47 authored by Simon Praetorius's avatar Simon Praetorius
Browse files

Merge branch 'feature/have-mpi' into 'master'

Handle optional dependency MPI by directly passing HAVE_MPI flag

See merge request !1310
parents 76784f35 14bee5dd
Branches
Tags
1 merge request!1310Handle optional dependency MPI by directly passing HAVE_MPI flag
Pipeline #66346 failed
......@@ -24,7 +24,7 @@ set_package_properties("MPI" PROPERTIES
if(MPI_C_FOUND)
set(HAVE_MPI ${MPI_C_FOUND})
dune_register_package_flags(COMPILE_DEFINITIONS "ENABLE_MPI=1"
dune_register_package_flags(COMPILE_DEFINITIONS "HAVE_MPI=1"
LIBRARIES MPI::MPI_C)
endif()
......@@ -36,7 +36,7 @@ function(add_dune_mpi_flags)
if(MPI_C_FOUND)
foreach(target ${targets})
target_link_libraries(${target} PUBLIC MPI::MPI_C)
target_compile_definitions(${target} PUBLIC "ENABLE_MPI=1")
target_compile_definitions(${target} PUBLIC "HAVE_MPI=1")
endforeach(target)
endif()
endfunction(add_dune_mpi_flags)
......@@ -36,9 +36,6 @@
/* Define if you have LAPACK library. */
#cmakedefine HAVE_LAPACK 1
/* Define if you have the MPI library. */
#cmakedefine HAVE_MPI ENABLE_MPI
/* Deactivate cxx bindings for MPI */
#if defined(HAVE_MPI) && HAVE_MPI
#define MPICH_SKIP_MPICXX 1
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment