Skip to content

Commit

Permalink
Remove global include header
Browse files Browse the repository at this point in the history
  • Loading branch information
majetideepak committed Sep 12, 2024
1 parent d02c3df commit c51fdc4
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,6 @@ endif()
if(DEFINED ENV{INSTALL_PREFIX})
message(STATUS "Dependency install directory set to: $ENV{INSTALL_PREFIX}")
list(APPEND CMAKE_PREFIX_PATH "$ENV{INSTALL_PREFIX}")
# Allow installed package headers to be picked up before brew/system package
# headers
include_directories(BEFORE "$ENV{INSTALL_PREFIX}/include")
endif()

list(PREPEND CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/CMake"
Expand Down Expand Up @@ -420,6 +417,10 @@ endif()

set_source(fmt)
resolve_dependency(fmt 9.0.0)
get_target_property(fmt_INCLUDE_DIRS fmt::fmt INTERFACE_INCLUDE_DIRECTORIES)
message(STATUS "FMT header path: ${fmt_INCLUDE_DIRS}")
# Prevent brew fmt headers from taking precedence.
include_directories(BEFORE ${fmt_INCLUDE_DIRS})

if(${VELOX_BUILD_MINIMAL_WITH_DWIO} OR ${VELOX_ENABLE_HIVE_CONNECTOR})
# DWIO needs all sorts of stream compression libraries.
Expand Down

0 comments on commit c51fdc4

Please sign in to comment.