This is an automated email from the ASF dual-hosted git repository.
zhouyuan 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 4566fe034f [VL] Fix dynamic link on rhel (#12359)
4566fe034f is described below
commit 4566fe034ffe3b468266f819ea5e64f7175c5dad
Author: Rong Ma <[email protected]>
AuthorDate: Thu Jun 25 09:19:17 2026 +0100
[VL] Fix dynamic link on rhel (#12359)
---
cpp/velox/CMakeLists.txt | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/cpp/velox/CMakeLists.txt b/cpp/velox/CMakeLists.txt
index b267aa6e95..4ec3f9f0f2 100644
--- a/cpp/velox/CMakeLists.txt
+++ b/cpp/velox/CMakeLists.txt
@@ -294,11 +294,6 @@ target_include_directories(
set_target_properties(velox PROPERTIES LIBRARY_OUTPUT_DIRECTORY
${root_directory}/releases)
-# If folly is not installed in system lib paths, please add
-# `-DCMAKE_PREFIX_PATH="${folly lib path}" to cmake arguments. It is also
-# applicable to other dependencies.
-find_package(Folly REQUIRED CONFIG)
-
if(ENABLE_JEMALLOC_STATS)
include(Findjemalloc)
find_jemalloc()
@@ -345,7 +340,6 @@ endif()
target_link_libraries(velox PUBLIC facebook::velox)
-target_link_libraries(velox PUBLIC Folly::folly)
target_link_libraries(velox PUBLIC ${GLUTEN_ROARING_LINK_LIBRARY})
find_re2()
@@ -388,6 +382,12 @@ else()
endif()
target_link_libraries(velox PUBLIC external::veloxthrift)
+# If folly is not installed in system lib paths, please add
+# `-DCMAKE_PREFIX_PATH="${folly lib path}" to cmake arguments. It is also
+# applicable to other dependencies.
+find_package(Folly REQUIRED CONFIG)
+target_link_libraries(velox PUBLIC Folly::folly)
+
# Link thrift libraries - check vcpkg first, then fall back to system libraries
# Determine vcpkg triplet directory based on architecture
if(CMAKE_SYSTEM_PROCESSOR MATCHES "(x86)|(X86)|(amd64)|(AMD64)")
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]