Skip to content

Commit 65c7048

Browse files
committed
cmake: Suppress compiler warnings from capnproto headers
Tweak target_link_libraries calls to correctly declare dependencies on capnproto libraries as public, so capnproto include directories get correctly added to downstream targets. Without this, compiler warnings can be triggered from capnproto headers because they are not treated like external headers. Fixes #138
1 parent 1e06ff0 commit 65c7048

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

CMakeLists.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,10 +95,10 @@ target_include_directories(multiprocess PUBLIC
9595
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
9696
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
9797
${CAPNP_INCLUDE_DIRECTORY})
98-
target_link_libraries(multiprocess PRIVATE CapnProto::capnp)
99-
target_link_libraries(multiprocess PRIVATE CapnProto::capnp-rpc)
100-
target_link_libraries(multiprocess PRIVATE CapnProto::kj)
101-
target_link_libraries(multiprocess PRIVATE CapnProto::kj-async)
98+
target_link_libraries(multiprocess PUBLIC CapnProto::capnp)
99+
target_link_libraries(multiprocess PUBLIC CapnProto::capnp-rpc)
100+
target_link_libraries(multiprocess PUBLIC CapnProto::kj)
101+
target_link_libraries(multiprocess PUBLIC CapnProto::kj-async)
102102
set_target_properties(multiprocess PROPERTIES
103103
PUBLIC_HEADER "${MP_PUBLIC_HEADERS}")
104104
install(TARGETS multiprocess EXPORT LibTargets

0 commit comments

Comments
 (0)