Skip to content
Snippets Groups Projects
Commit 544adf6b authored by Deepak Rangarajan's avatar Deepak Rangarajan Committed by Mark Meredith
Browse files

switch cmake to use target_compile_definitions instead of add_definitions

parent af511efb
No related branches found
No related tags found
1 merge request!105switch cmake to use target_compile_definitions instead of add_definitions
Pipeline #29064 passed
Stage: build-img
Stage: test
Hide whitespace changes
Inline Side-by-side
Showing
with 4 additions and 1 deletion
CMakeLists.txt
+ 4
− 1
View file @ 544adf6b
...@@ -23,7 +23,6 @@ find_package(pegtl REQUIRED) ...@@ -23,7 +23,6 @@ find_package(pegtl REQUIRED)
if (CSG_CGAL_ENABLED) if (CSG_CGAL_ENABLED)
find_package(CGAL REQUIRED) find_package(CGAL REQUIRED)
add_definitions(-DUSE_CGAL)
endif() endif()
add_subdirectory(src/csg) add_subdirectory(src/csg)
...@@ -34,3 +33,7 @@ target_include_directories(csg-eb ...@@ -34,3 +33,7 @@ target_include_directories(csg-eb
) )
target_compile_features(csg-eb PRIVATE cxx_std_17) target_compile_features(csg-eb PRIVATE cxx_std_17)
if (CSG_CGAL_ENABLED)
target_compile_definitions(csg-eb PUBLIC USE_CGAL)
endif()
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment