Skip to content
Snippets Groups Projects
Commit 13411d62 authored by Christoph Grüninger's avatar Christoph Grüninger
Browse files

[!736] cmake: separate commandline args to list

Merge branch 'fix/mpi-cmake' into 'master'

ref:core/dune-common Providing space-separated MPI_DUNE_COMPILE_FLAGS would
previously add them as a single list entry to the cmake list COMPILE_OPTIONS
and thus ruin the command-line having something like

    /usr/bin/c++ "-arg1 -arg2" -other -args

Supersedes [!709]

See merge request [!736]

  [!709]: gitlab.dune-project.org/NoneNone/merge_requests/709
  [!736]: gitlab.dune-project.org/core/dune-common/merge_requests/736
parents 5304152a 81fc5be3
No related branches found
No related tags found
1 merge request!736cmake: separate commandline args to list
Pipeline #23078 passed
......@@ -58,7 +58,8 @@ function(add_dune_mpi_flags)
set(_prefix TARGET)
endif()
if(MPI_C_FOUND)
set_property(${_prefix} ${ADD_MPI_UNPARSED_ARGUMENTS} APPEND PROPERTY COMPILE_OPTIONS ${MPI_DUNE_COMPILE_FLAGS})
separate_arguments(MPI_DUNE_COMPILE_FLAGS_LIST UNIX_COMMAND ${MPI_DUNE_COMPILE_FLAGS})
set_property(${_prefix} ${ADD_MPI_UNPARSED_ARGUMENTS} APPEND PROPERTY COMPILE_OPTIONS ${MPI_DUNE_COMPILE_FLAGS_LIST})
set_property(${_prefix} ${ADD_MPI_UNPARSED_ARGUMENTS} APPEND PROPERTY COMPILE_DEFINITIONS ENABLE_MPI=1
MPICH_SKIP_MPICXX MPIPP_H)
if(NOT (ADD_MPI_SOURCE_ONLY OR ADD_MPI_OBJECT))
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment