(gluten) branch main updated: [VL] Fix macOS libvelox linking with Velox dbgen (#12530)

[mi...@apache.org]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 580835de86 [VL] Fix macOS libvelox linking with Velox dbgen (#12530)
580835de86 is described below

commit 580835de86cc18058bba94a0a709c00cd91d28ba
Author: Mingliang Zhu <zh...@gmail.com>
AuthorDate: Thu Jul 16 16:25:50 2026 +0800

    [VL] Fix macOS libvelox linking with Velox dbgen (#12530)
---
 cpp/velox/CMakeLists.txt | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/cpp/velox/CMakeLists.txt b/cpp/velox/CMakeLists.txt
index 636b073045..78baafef8f 100644
--- a/cpp/velox/CMakeLists.txt
+++ b/cpp/velox/CMakeLists.txt
@@ -323,12 +323,14 @@ if(CMAKE_SYSTEM_NAME MATCHES "Darwin")
   set(GLUTEN_VELOX_LINK_SCOPE PRIVATE)
 endif()
 
-if(BUILD_TESTS)
-  add_duckdb()
-
+if(BUILD_TESTS OR CMAKE_SYSTEM_NAME MATCHES "Darwin")
   import_library(facebook::velox::dbgen
                  ${VELOX_BUILD_PATH}/velox/tpch/gen/dbgen/libdbgen.a)
   target_link_libraries(facebook::velox INTERFACE facebook::velox::dbgen)
+endif()
+
+if(BUILD_TESTS)
+  add_duckdb()
 
   import_library(
     facebook::velox::vector_test_lib


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org