Skip to content

Commit 0d31976

Browse files
committed
fix(sync): make vix_warnings and sanitizers private
avoid exporting build-only helpers
1 parent 51a8b9a commit 0d31976

1 file changed

Lines changed: 3 additions & 7 deletions

File tree

CMakeLists.txt

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -131,9 +131,10 @@ if (SYNC_SOURCES)
131131

132132
if (TARGET vix_warnings)
133133
message(STATUS "[sync] vix_warnings detected (from umbrella)")
134-
target_link_libraries(vix_sync PUBLIC vix_warnings)
134+
target_link_libraries(vix_sync PRIVATE vix_warnings)
135+
135136
if (VIX_ENABLE_SANITIZERS AND TARGET vix_sanitizers)
136-
target_link_libraries(vix_sync PUBLIC vix_sanitizers)
137+
target_link_libraries(vix_sync PRIVATE vix_sanitizers)
137138
endif()
138139

139140
else()
@@ -192,11 +193,6 @@ else()
192193
${VIX_JSON_TARGET}
193194
)
194195

195-
196-
if (VIX_ENABLE_SANITIZERS AND TARGET vix_sanitizers)
197-
target_link_libraries(vix_sync INTERFACE vix_sanitizers)
198-
endif()
199-
200196
install(TARGETS vix_sync
201197
EXPORT VixTargets
202198
INCLUDES DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}

0 commit comments

Comments
 (0)