Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Enable folly #293

Merged
merged 8 commits into from
Mar 21, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Use modern cmake targets for folly
alexbaden committed Mar 20, 2023
commit 7a7b2be062c6f00c9625089ab47c2a7e300e2dca
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -94,9 +94,9 @@ if(ENABLE_FOLLY)
if(NOT Folly_FOUND)
set(ENABLE_FOLLY OFF CACHE BOOL "Use Folly" FORCE)
else()
include_directories(${Folly_INCLUDE_DIRS})
set(FOLLY_LIBRARIES "")
add_definitions("-DHAVE_FOLLY")
list(APPEND Folly_LIBRARIES ${Glog_LIBRARIES})
list(APPEND Folly_LIBRARIES Folly::folly ${Glog_LIBRARIES})
# TODO: use Folly::folly_deps?
if(MSVC)
find_package(Libevent COMPONENTS core REQUIRED)
89 changes: 0 additions & 89 deletions cmake_modules/FindFolly.cmake

This file was deleted.

4 changes: 2 additions & 2 deletions omniscidb/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -313,9 +313,9 @@ if(ENABLE_FOLLY)
if(NOT Folly_FOUND)
set(ENABLE_FOLLY OFF CACHE BOOL "Use Folly" FORCE)
else()
include_directories(${Folly_INCLUDE_DIRS})
set(FOLLY_LIBRARIES "")
add_definitions("-DHAVE_FOLLY")
list(APPEND Folly_LIBRARIES ${Glog_LIBRARIES})
list(APPEND Folly_LIBRARIES Folly::folly ${Glog_LIBRARIES})
# TODO: use Folly::folly_deps?
if(MSVC)
find_package(Libevent COMPONENTS core REQUIRED)