Skip to content

Commit 6c3ae14

Browse files
committed
Use private linking in CMake for MPI libs
1 parent 5a3096a commit 6c3ae14

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

mpi/mpi-caliquery/CMakeLists.txt

+1
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,6 @@ add_executable(mpi-caliquery
55
mpi-caliquery.cpp)
66

77
target_link_libraries(mpi-caliquery caliper-mpi caliper-mpi-common caliper-tools-util caliper)
8+
target_link_libraries(mpi-caliquery ${MPI_CXX_LIBRARIES})
89

910
install(TARGETS mpi-caliquery DESTINATION ${CMAKE_INSTALL_BINDIR})

mpi/mpi-common/CMakeLists.txt

+2-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ add_library(caliper-mpi-common ${CALIPER_MPI_SOURCES})
66
set_target_properties(caliper-mpi-common PROPERTIES SOVERSION ${CALIPER_MAJOR_VERSION})
77
set_target_properties(caliper-mpi-common PROPERTIES VERSION ${CALIPER_VERSION})
88

9-
target_link_libraries(caliper-mpi-common caliper-reader)
10-
target_link_libraries(caliper-mpi-common ${MPI_C_LIBRARIES} ${MPI_CXX_LIBRARIES})
9+
target_link_libraries(caliper-mpi-common PUBLIC caliper-reader)
10+
target_link_libraries(caliper-mpi-common PRIVATE ${MPI_C_LIBRARIES} ${MPI_CXX_LIBRARIES})
1111

1212
install(TARGETS caliper-mpi-common
1313
EXPORT caliper

mpi/mpi-rt/CMakeLists.txt

+2-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ add_library(caliper-mpi
4242
set_target_properties(caliper-mpi PROPERTIES SOVERSION ${CALIPER_MAJOR_VERSION})
4343
set_target_properties(caliper-mpi PROPERTIES VERSION ${CALIPER_VERSION})
4444

45-
target_link_libraries(caliper-mpi caliper-mpi-common caliper)
45+
target_link_libraries(caliper-mpi PUBLIC caliper-mpi-common caliper)
46+
target_link_libraries(caliper-mpi PRIVATE ${MPI_CXX_LIBRARIES})
4647

4748
install(TARGETS caliper-mpi
4849
EXPORT caliper

0 commit comments

Comments
 (0)