This is an automated email from the ASF dual-hosted git repository.

philo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new d1eca82d3 [GLUTEN-6529][VL] Fix build error on macOS caused by 
ConfigArrow.cmake (#6530)
d1eca82d3 is described below

commit d1eca82d30418d08600b865770de9e786a9c2944
Author: James Xu <[email protected]>
AuthorDate: Mon Jul 22 09:57:58 2024 +0800

    [GLUTEN-6529][VL] Fix build error on macOS caused by ConfigArrow.cmake 
(#6530)
---
 cpp/CMake/ConfigArrow.cmake | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/cpp/CMake/ConfigArrow.cmake b/cpp/CMake/ConfigArrow.cmake
index 1de4ba406..ebef7abd6 100644
--- a/cpp/CMake/ConfigArrow.cmake
+++ b/cpp/CMake/ConfigArrow.cmake
@@ -44,7 +44,21 @@ function(FIND_ARROW_LIB LIB_NAME)
       message(FATAL_ERROR "Arrow library Not Found: ${ARROW_LIB_FULL_NAME}")
     endif()
     message(STATUS "Found Arrow library: ${ARROW_LIB_${LIB_NAME}}")
-    set_target_properties(Arrow::${LIB_NAME}
-                          PROPERTIES IMPORTED_LOCATION 
${ARROW_LIB_${LIB_NAME}})
+
+    # Get the parent-parent directory of the lib file. For example:
+    #
+    # * ${ARROW_LIB_${LIB_NAME}}: /usr/local/lib/libarrow.a
+    # * ${ARROW_LIB_INCLUDE_DIR}: /usr/local
+    #
+    # Then we can get our include directory: /usr/local/include
+    get_filename_component(ARROW_LIB_INCLUDE_DIR "${ARROW_LIB_${LIB_NAME}}"
+                           PATH)
+    get_filename_component(ARROW_LIB_INCLUDE_DIR "${ARROW_LIB_INCLUDE_DIR}"
+                           PATH)
+
+    set_target_properties(
+      Arrow::${LIB_NAME}
+      PROPERTIES IMPORTED_LOCATION ${ARROW_LIB_${LIB_NAME}}
+                 INTERFACE_INCLUDE_DIRECTORIES 
${ARROW_LIB_INCLUDE_DIR}/include)
   endif()
 endfunction()


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to