[coco.core] Explicitly specify link visibility (#2978)
author박종현/On-Device Lab(SR)/Staff Engineer/삼성전자 <jh1302.park@samsung.com>
Fri, 1 Feb 2019 01:43:21 +0000 (10:43 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Fri, 1 Feb 2019 01:43:21 +0000 (10:43 +0900)
This commit specifies link visibility over dependent libraries.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
contrib/coco/core/CMakeLists.txt

index ddbdaa2..0eafe44 100644 (file)
@@ -4,9 +4,9 @@ list(REMOVE_ITEM SOURCES ${TESTS})
 
 add_library(coco_core SHARED ${SOURCES})
 target_include_directories(coco_core PUBLIC include)
-target_link_libraries(coco_core nncc_core)
-# TODO Set this as PRIVATE dependency
-target_link_libraries(coco_core stdex)
+# NOTE Some coco_core PUBLIC headers include nncc_core headers
+target_link_libraries(coco_core PUBLIC nncc_core)
+target_link_libraries(coco_core PRIVATE stdex)
 
 nncc_find_package(GTest QUIET)
 
@@ -18,3 +18,4 @@ nncc_include(OptionalTargetTools)
 
 coco_add_test(coco_core_test ${TESTS})
 optional_target_link_libraries(coco_core_test coco_core)
+optional_target_link_libraries(coco_core_test stdex)