From 2800c5f100916f295cbc302a0a2862cf248b562e Mon Sep 17 00:00:00 2001 From: Sutou Kouhei Date: Tue, 30 Aug 2022 17:47:41 +0900 Subject: [PATCH] ARROW-17565: [C++] Backward compatible ${PACKAGE}_shared CMake target isn't provided This is a follow-up of ARROW-12175 / #13892 . We introduced `${PACKAGE}::` namespace to all exported CMake targets such as `Arrow::arrow_shared` and `Arrow::arrow_static` but we also provides no namespaced CMake targets such as `arrow_share` and `arrow_static` as aliases of namespaced CMake targets. But the backward compatibility feature isn't worked for `_shared`. --- cpp/src/arrow/ArrowConfig.cmake.in | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cpp/src/arrow/ArrowConfig.cmake.in b/cpp/src/arrow/ArrowConfig.cmake.in index 83a0f849c48..0aa298b6658 100644 --- a/cpp/src/arrow/ArrowConfig.cmake.in +++ b/cpp/src/arrow/ArrowConfig.cmake.in @@ -108,7 +108,8 @@ macro(arrow_keep_backward_compatibility namespace target_base_name) string(TOUPPER ${target_base_name} target_base_name_upper) if(NOT CMAKE_VERSION VERSION_LESS 3.18) - if(TARGET ${namespace}::${target_base_name} AND NOT TARGET ${target_base_name}_shared) + if(TARGET ${namespace}::${target_base_name}_shared AND NOT TARGET + ${target_base_name}_shared) add_library(${target_base_name}_shared ALIAS ${namespace}::${target_base_name}_shared) endif()