Explain why no find_package(MPI) in tests

Applying Carl's suggestion.
kokkos · Jun 12, 2024 · f869f73 · f869f73
1 parent 0f0892a
commit f869f73
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/perf_tests/CMakeLists.txt b/perf_tests/CMakeLists.txt
@@ -8,8 +8,8 @@ enable_testing()
 if (NOT TARGET KokkosComm::KokkosComm)
   find_package(KokkosComm REQUIRED)
 
-  # Find MPI, useful for standalone tests and for having MPI_EXEC macros
-  # find_package(MPI REQUIRED) # not needed, KokkosComm already does this
+  # find_package(MPI REQUIRED), useful for standalone tests and for having MPI_EXEC macros
+  # Commented out because KokkosComm already does this and we want to test the export is working properly
 endif()
 
 include(FetchContent)

diff --git a/unit_tests/CMakeLists.txt b/unit_tests/CMakeLists.txt
@@ -8,8 +8,8 @@ enable_testing()
 if (NOT TARGET KokkosComm::KokkosComm)
   find_package(KokkosComm REQUIRED)
 
-  # Find MPI, useful for standalone tests and for having MPI_EXEC macros
-  # find_package(MPI REQUIRED) # not needed, KokkosComm already does this
+  # find_package(MPI REQUIRED), useful for standalone tests and for having MPI_EXEC macros
+  # Commented out because KokkosComm already does this and we want to test the export is working properly
 endif()
 
 include(FetchContent)