Skip to content

Commit ab2ee63

Browse files
MBkktLakehouse Engine Bot
authored andcommitted
fix: Use explicit cmake dependency for fastfloat instead of implicit from folly
Alchemy-item: (ID = 795) fix: Use explicit cmake dependency for fastfloat instead of implicit from folly commit 1/1 - 2443339
1 parent a070edc commit ab2ee63

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

CMake/resolve_dependency_modules/folly/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414
project(Folly)
1515
cmake_minimum_required(VERSION 3.28)
1616

17-
velox_set_source(fastfloat)
18-
velox_resolve_dependency(fastfloat CONFIG REQUIRED)
17+
velox_set_source(FastFloat)
18+
velox_resolve_dependency(FastFloat CONFIG REQUIRED)
1919

2020
set(VELOX_FOLLY_BUILD_VERSION v2025.04.28.00)
2121
set(

CMakeLists.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -618,6 +618,9 @@ endif()
618618
velox_set_source(simdjson)
619619
velox_resolve_dependency(simdjson 3.13.0)
620620

621+
velox_set_source(FastFloat)
622+
velox_resolve_dependency(FastFloat)
623+
621624
velox_set_source(folly)
622625
velox_resolve_dependency(folly)
623626

velox/functions/prestosql/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ velox_link_libraries(
7575
velox_presto_types
7676
velox_functions_util
7777
Folly::folly
78+
FastFloat::fast_float
7879
stemmer::stemmer
7980
)
8081

0 commit comments

Comments
 (0)