Skip to content

Commit

Permalink
[VL] Allow shared dependencies for s3 and abfs libs (#8402)
Browse files Browse the repository at this point in the history
  • Loading branch information
majetideepak authored Jan 3, 2025
1 parent e5e2285 commit 5fb0448
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions cpp/velox/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -112,10 +112,7 @@ macro(find_re2)
endmacro()

macro(find_awssdk)
set(CMAKE_FIND_LIBRARY_SUFFIXES_BCK ${CMAKE_FIND_LIBRARY_SUFFIXES})
set(CMAKE_FIND_LIBRARY_SUFFIXES ".a")
find_package(AWSSDK REQUIRED COMPONENTS s3;identity-management)
set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES_BCK})
endmacro()

macro(find_gcssdk)
Expand All @@ -125,12 +122,9 @@ endmacro()
macro(find_azure)
find_package(CURL REQUIRED)
find_package(LibXml2 REQUIRED)
set(CMAKE_FIND_LIBRARY_SUFFIXES_BCK ${CMAKE_FIND_LIBRARY_SUFFIXES})
set(CMAKE_FIND_LIBRARY_SUFFIXES ".a")
find_package(azure-storage-blobs-cpp CONFIG REQUIRED)
find_package(azure-storage-files-datalake-cpp CONFIG REQUIRED)
find_package(azure-identity-cpp CONFIG REQUIRED)
set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_FIND_LIBRARY_SUFFIXES_BCK})
endmacro()

# Build Velox backend.
Expand Down

0 comments on commit 5fb0448

Please sign in to comment.