[jira] [Resolved] (ARROW-16751) [C++] Add support for building bundled ucx with CMake 3.5

["Kouhei Sutou (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ARROW-16751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kouhei Sutou resolved ARROW-16751.
----------------------------------
    Resolution: Fixed

Issue resolved by pull request 13389
[https://github.com/apache/arrow/pull/13389]

> [C++] Add support for building bundled ucx with CMake 3.5
> ---------------------------------------------------------
>
>                 Key: ARROW-16751
>                 URL: https://issues.apache.org/jira/browse/ARROW-16751
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Yibo Cai
>            Assignee: Yibo Cai
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 9.0.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Context: [https://github.com/apache/arrow/pull/13244#discussion_r889780669]
> {{target_include_directories()}} in cmake 3.10 or earlier doesn't support {{INTERFACE}} against {{IMPORTED}} target, so we have to check cmake version like below:
> {code:java}
> if(CMAKE_VERSION VERSION_LESS 3.11)
>   set_target_properties(xsimd PROPERTIES INTERFACE_INCLUDE_DIRECTORIES
>                                          "${XSIMD_INCLUDE_DIR}")
> else()
>   target_include_directories(xsimd INTERFACE "${XSIMD_INCLUDE_DIR}")
> endif()
> {code}
> Above code is duplicated for some targets. There are also some targets (e.g. ucx::ucx) missed the check.
> We can add a function {{arrow_imported_target_interface_include_directories()}} to make it simpler.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)