[VL][Delta] Fix roaring link propagation

Author: Mohammad Linjawi

cpp/CMake/FindRoaring.cmake

@@ -61,9 +61,18 @@ function(_gluten_roaring_add_headers target_name)
   endif()
 endfunction()
 
+function(_gluten_roaring_ensure_target)
+  if(NOT TARGET gluten_roaring)
+    add_library(gluten_roaring INTERFACE)
+    add_library(gluten::roaring ALIAS gluten_roaring)
+  endif()
+endfunction()
+
 # Check if roaring target already exists.
 if(TARGET roaring)
-  _gluten_roaring_add_headers(roaring)
+  _gluten_roaring_ensure_target()
+  target_link_libraries(gluten_roaring INTERFACE roaring)
+  _gluten_roaring_add_headers(gluten_roaring)
   message(STATUS "Target roaring was already found.")
   return()
 endif()
@@ -86,9 +95,9 @@ if(PkgConfig_FOUND)
 endif()
 
 if(Roaring_FOUND)
-  add_library(roaring INTERFACE)
-  target_link_libraries(roaring INTERFACE PkgConfig::Roaring)
-  _gluten_roaring_add_headers(roaring)
+  _gluten_roaring_ensure_target()
+  target_link_libraries(gluten_roaring INTERFACE PkgConfig::Roaring)
+  _gluten_roaring_add_headers(gluten_roaring)
   message(STATUS "Found roaring via pkg-config.")
   return()
 endif()
@@ -109,13 +118,11 @@ find_library(
 if(Roaring_INCLUDE_DIR
    AND Roaring_CPP_INCLUDE_DIR
    AND Roaring_LIBRARY)
-  add_library(roaring UNKNOWN IMPORTED)
-  set_target_properties(
-    roaring
-    PROPERTIES IMPORTED_LOCATION "${Roaring_LIBRARY}"
-               INTERFACE_INCLUDE_DIRECTORIES
-               "${Roaring_INCLUDE_DIR};${Roaring_CPP_INCLUDE_DIR}")
-  _gluten_roaring_add_headers(roaring)
+  _gluten_roaring_ensure_target()
+  target_link_libraries(gluten_roaring INTERFACE "${Roaring_LIBRARY}")
+  target_include_directories(
+    gluten_roaring INTERFACE "${Roaring_INCLUDE_DIR}"
+                             "${Roaring_CPP_INCLUDE_DIR}")
   message(STATUS "Found roaring via direct library lookup.")
   return()
 endif()
@@ -143,7 +150,12 @@ FetchContent_MakeAvailable(roaring_fetch)
 
 if(TARGET roaring)
   set_target_properties(roaring PROPERTIES POSITION_INDEPENDENT_CODE ON)
-  _gluten_roaring_add_headers(roaring)
+  _gluten_roaring_ensure_target()
+  target_link_libraries(gluten_roaring INTERFACE "$<TARGET_FILE:roaring>")
+  target_include_directories(
+    gluten_roaring INTERFACE "${roaring_fetch_SOURCE_DIR}/include"
+                             "${roaring_fetch_SOURCE_DIR}/cpp")
+  add_dependencies(gluten_roaring roaring)
   if(_roaring_restore_pic)
     set(CMAKE_POSITION_INDEPENDENT_CODE "${_roaring_saved_pic}")
   else()

cpp/velox/CMakeLists.txt

@@ -326,7 +326,7 @@ endif()
 target_link_libraries(velox PUBLIC facebook::velox)
 
 target_link_libraries(velox PUBLIC Folly::folly)
-target_link_libraries(velox PUBLIC roaring)
+target_link_libraries(velox PUBLIC gluten::roaring)
 
 find_re2()
 target_link_libraries(velox PUBLIC ${RE2_LIBRARY})