Skip to content

Commit

Permalink
Fix duckdb build failure
Browse files Browse the repository at this point in the history
Upgrade FBOS dependencies to 2024.09.16.00 installs all dependencies in
deps-install subfolder, and set the include directory to it in front of
other include directories. This caused duckdb build failed with "error:
use of undeclared identifier 'FMT_SNPRINTF'". This commit removes the
BEFORE keyword to fix the problem.
  • Loading branch information
yingsu00 committed Sep 23, 2024
1 parent b78bb3a commit 40a18d8
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,14 @@ if(DEFINED ENV{CONDA_PREFIX})
endif()
endif()

message(STATUS "INSTALL_PREFIX $ENV{INSTALL_PREFIX}")
if(DEFINED ENV{INSTALL_PREFIX})
message(STATUS "Dependency install directory set to: $ENV{INSTALL_PREFIX}")
list(APPEND CMAKE_PREFIX_PATH "$ENV{INSTALL_PREFIX}")
message(STATUS "CMAKE_PREFIX_PATH ${CMAKE_PREFIX_PATH}")
# Allow installed package headers to be picked up before brew/system package
# headers
include_directories(BEFORE "$ENV{INSTALL_PREFIX}/include")
# include_directories("$ENV{INSTALL_PREFIX}/include")
endif()

list(PREPEND CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/CMake"
Expand Down

0 comments on commit 40a18d8

Please sign in to comment.