From a9fd0a3729430f36ada0e53657fdb8c5d48cdd4c Mon Sep 17 00:00:00 2001 From: Laurent Rineau Date: Fri, 26 Apr 2019 17:12:01 +0200 Subject: [PATCH] Bug-fix: Boost Thread is a dependency of CGAL_Core only for GNU/g++ Previously, it was documented to g++ only, but the code was just checking "not MSVC", and thus clang was in the set. --- .../cmake/modules/CGAL_SetupCGAL_CoreDependencies.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Installation/cmake/modules/CGAL_SetupCGAL_CoreDependencies.cmake b/Installation/cmake/modules/CGAL_SetupCGAL_CoreDependencies.cmake index e22308e0025..f5c7c4945b7 100644 --- a/Installation/cmake/modules/CGAL_SetupCGAL_CoreDependencies.cmake +++ b/Installation/cmake/modules/CGAL_SetupCGAL_CoreDependencies.cmake @@ -71,8 +71,8 @@ function(CGAL_setup_CGAL_Core_dependencies target) target_link_libraries( CGAL_Core ${keyword} CGAL::CGAL ) # See the release notes of CGAL-4.10: CGAL_Core now requires - # Boost.Thread, with all compilers but MSVC. - if (NOT MSVC) + # Boost.Thread, with GNU G++. + if (CMAKE_CXX_COMPILER_ID STREQUAL GNU) if(TARGET Boost::thread) target_link_libraries( CGAL_Core ${keyword} Boost::thread) else()