Refactor

apache · Aug 7, 2024 · e528c20 · e528c20
1 parent ed8acce
commit e528c20
Showing 1 changed file with 9 additions and 16 deletions.
diff --git a/cpp/velox/CMakeLists.txt b/cpp/velox/CMakeLists.txt
@@ -99,19 +99,13 @@ function(import_library TARGET_NAME LIB_PATH)
   set_target_properties(${TARGET_NAME} PROPERTIES IMPORTED_LOCATION ${LIB_PATH})
 endfunction()
 
-function(add_velox_dependency VELOX_DEP_LIB_NAME VELOX_DEP_LIB_PATH)
-  set(VELOX_DEP_LIB facebook::velox::${VELOX_DEP_LIB_NAME})
-  import_library(${VELOX_DEP_LIB} ${VELOX_DEP_LIB_PATH})
-  target_link_libraries(velox PUBLIC ${VELOX_DEP_LIB})
-endfunction()
-
 macro(add_duckdb)
   find_package(DuckDB)
   if(NOT DuckDB_FOUND)
     message(FATAL_ERROR "Cannot find DuckDB.")
   else()
     message(STATUS "Found DuckDB library from ${DuckDB_DIR}")
-    target_link_libraries(velox PUBLIC duckdb_static)
+    target_link_libraries(facebook::velox INTERFACE duckdb_static)
   endif()
 endmacro()
 
@@ -276,6 +270,8 @@ set_target_properties(
                              ${VELOX_BUILD_PATH}/lib/libvelox.a)
 
 if(BUILD_TESTS)
+  add_duckdb()
+
   import_library(facebook::velox::dbgen
                  ${VELOX_BUILD_PATH}/velox/tpch/gen/dbgen/libdbgen.a)
   target_link_libraries(facebook::velox INTERFACE facebook::velox::dbgen)
@@ -311,27 +307,24 @@ endif()
 
 target_link_libraries(velox PUBLIC facebook::velox)
 
-if(BUILD_TESTS)
-  add_duckdb()
-endif()
-
 target_link_libraries(velox PUBLIC Folly::folly)
 
 find_re2()
 target_link_libraries(velox PUBLIC ${RE2_LIBRARY})
 
-add_velox_dependency(
-  external::stemmer
-  "${VELOX_BUILD_PATH}/_deps/libstemmer/src/libstemmer/libstemmer.a")
+import_library(external::stemmer
+               ${VELOX_BUILD_PATH}/_deps/libstemmer/src/libstemmer/libstemmer.a)
+target_link_libraries(velox PUBLIC external::stemmer)
 
 set(CMAKE_FIND_LIBRARY_SUFFIXES_BCK ${CMAKE_FIND_LIBRARY_SUFFIXES})
 set(CMAKE_FIND_LIBRARY_SUFFIXES ".a")
 find_package(simdjson CONFIG)
 if(simdjson_FOUND AND TARGET simdjson::simdjson)
   target_link_libraries(velox PUBLIC simdjson::simdjson)
 else()
-  add_velox_dependency(external::simdjson
-                       "${VELOX_BUILD_PATH}/_deps/simdjson-build/libsimdjson.a")
+  import_library(external::simdjson
+                 ${VELOX_BUILD_PATH}/_deps/simdjson-build/libsimdjson.a)
+  target_link_libraries(velox PUBLIC external::simdjson)
 endif()
 set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES_BCK})