Skip to content

Commit 245b331

Browse files
author
Roman Dubtsov
committed
build: do not break transitive linking when being used as a subproject
1 parent 9670998 commit 245b331

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

cmake/utils.cmake

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,15 +105,15 @@ macro(append_to_windows_path_list path_list path)
105105
endif()
106106
endmacro()
107107

108-
function(target_link_libraries_private target list)
108+
function(target_link_libraries_build target list)
109109
# Foreach is required for compatibility with 2.8.11 ways
110110
foreach(lib ${list})
111-
target_link_libraries(${target} LINK_PRIVATE
111+
target_link_libraries(${target}
112112
"$<BUILD_INTERFACE:${lib}>")
113113
endforeach(lib)
114114
endfunction()
115115

116-
function(target_link_libraries_public target list)
116+
function(target_link_libraries_install target list)
117117
# Foreach is required for compatibility with 2.8.11 ways
118118
foreach(lib ${list})
119119
get_filename_component(base "${lib}" NAME)

src/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,11 +103,11 @@ target_include_directories(${LIB_NAME} PUBLIC
103103
$<INSTALL_INTERFACE:$<INSTALL_PREFIX>/${CMAKE_INSTALL_INCLUDEDIR}>
104104
)
105105

106-
target_link_libraries_private(${LIB_NAME}
106+
target_link_libraries_build(${LIB_NAME}
107107
"${EXTRA_SHARED_LIBS};${EXTRA_STATIC_LIBS}")
108-
target_link_libraries_public(${LIB_NAME} "${EXTRA_SHARED_LIBS}")
108+
target_link_libraries_install(${LIB_NAME} "${EXTRA_SHARED_LIBS}")
109109
if(MKLDNN_LIBRARY_TYPE STREQUAL "STATIC")
110-
target_link_libraries_public(${LIB_NAME} "${EXTRA_STATIC_LIBS}")
110+
target_link_libraries_install(${LIB_NAME} "${EXTRA_STATIC_LIBS}")
111111
endif()
112112

113113
set(LIB_EXPORT_NAME "${LIB_NAME}-targets")

0 commit comments

Comments
 (0)