Skip to content

Commit

Permalink
Change the fix to add a define for boost only
Browse files Browse the repository at this point in the history
  • Loading branch information
Smjert committed Nov 21, 2023
1 parent 353e8eb commit b157caf
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
8 changes: 0 additions & 8 deletions cmake/flags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -254,11 +254,6 @@ function(setupBuildFlags)
"-weak_framework OSLog"
)

# This is a workaround for XCode 15 and boost
set(macos_cxx_defines
_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION
)

set(osquery_macos_common_defines
APPLE=1
DARWIN=1
Expand All @@ -277,9 +272,6 @@ function(setupBuildFlags)
target_link_libraries(cxx_settings INTERFACE
${macos_cxx_link_libraries}
)
target_compile_definitions(cxx_settings INTERFACE
${macos_cxx_defines}
)

list(APPEND osquery_defines ${osquery_macos_common_defines})
else()
Expand Down
4 changes: 4 additions & 0 deletions libraries/cmake/source/boost/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,10 @@ function(importBoostInterfaceLibraries)
importBoostInterfaceLibrary("${library_name}" ${library_dependency_list})
endforeach()

target_compile_definitions(thirdparty_boost_container_hash INTERFACE
BOOST_NO_CXX98_FUNCTION_BASE
)

# Additional settings for the libraries we just imported
if(PLATFORM_LINUX)
target_compile_definitions(thirdparty_boost_uuid INTERFACE
Expand Down

0 comments on commit b157caf

Please sign in to comment.