Correctly handle dependencies for SuperLUDist (TriBITSPub/TriBITS#299)

You only need to list a dependency on ParMETIS and not worry about METIS (which will come in through ParMETIS). This should fix one of the PR builds that enables SuperLUDist but not ParMETIS.
trilinos · Jul 15, 2022 · fa06795 · fa06795
1 parent af37943
commit fa06795
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cmake/TPLs/FindTPLSuperLUDist.cmake b/cmake/TPLs/FindTPLSuperLUDist.cmake
@@ -79,7 +79,7 @@ int main()
 "
   )
 
-  SET(CMAKE_REQUIRED_LIBRARIES SuperLUDist::all_libs ParMETIS::all_libs METIS::all_libs BLAS::all_libs)
+  SET(CMAKE_REQUIRED_LIBRARIES SuperLUDist::all_libs)
   SET(CMAKE_REQUIRED_FLAGS ${CMAKE_EXE_LINKER_FLAGS})
   CHECK_C_SOURCE_COMPILES("${SOURCE}" ${VARNAME})
 ENDFUNCTION()
@@ -100,7 +100,7 @@ int main()
 "
   )
 
-  SET(CMAKE_REQUIRED_LIBRARIES SuperLUDist::all_libs ParMETIS::all_libs METIS::all_libs BLAS::all_libs)
+  SET(CMAKE_REQUIRED_LIBRARIES SuperLUDist::all_libs)
   SET(CMAKE_REQUIRED_FLAGS ${CMAKE_EXE_LINKER_FLAGS})
   CHECK_C_SOURCE_COMPILES("${SOURCE}" ${VARNAME})
 ENDFUNCTION()

diff --git a/cmake/TPLs/FindTPLSuperLUDistDependencies.cmake b/cmake/TPLs/FindTPLSuperLUDistDependencies.cmake
@@ -1,2 +1,2 @@
 tribits_extpkg_define_dependencies( SuperLUDist
-  DEPENDENCIES  BLAS)
+  DEPENDENCIES  ParMETIS  BLAS)