Skip to content

Commit

Permalink
Revert "[CMake] Only replace find_package if required"
Browse files Browse the repository at this point in the history
This reverts commit 2a265a3.
  • Loading branch information
guitargeek committed Aug 21, 2024
1 parent e07f0aa commit 83b237d
Showing 1 changed file with 6 additions and 14 deletions.
20 changes: 6 additions & 14 deletions cmake/modules/SearchInstalledSoftware.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ include(FindPackageHandleStandardArgs)
set(lcgpackages http://lcgpackages.web.cern.ch/lcgpackages/tarFiles/sources)
string(REPLACE "-Werror " "" ROOT_EXTERNAL_CXX_FLAGS "${CMAKE_CXX_FLAGS} ")

macro(find_package)
if(NOT "${ARGV0}" IN_LIST ROOT_BUILTINS)
_find_package(${ARGV})
endif()
endmacro()

#---On MacOSX, try to find frameworks after standard libraries or headers------------
set(CMAKE_FIND_FRAMEWORK LAST)

Expand Down Expand Up @@ -1987,17 +1993,3 @@ endif()
if(test_distrdf_dask)
find_package(Dask 2022.08.1 REQUIRED)
endif()

#------------------------------------------------------------------------------------
# Modify find_package to ignore any ROOT builtins, such that the builtin LLVM
# doesn't find and use system versions of the builtin dependencies. This is a
# bit hacky, but fortunately this needs to be done only when builtins are used.
#
# See also discussion on GitHub: https://github.com/root-project/root/issues/8633
if(ROOT_BUILTINS)
macro(find_package)
if(NOT "${ARGV0}" IN_LIST ROOT_BUILTINS)
_find_package(${ARGV})
endif()
endmacro()
endif()

0 comments on commit 83b237d

Please sign in to comment.