Merge branch 'update/cmake' into 'main'

Add a find_package(Python3 required) to solve an installation issue

See merge request njoy/ACEtk!28

Author: whaeck

CMakeLists.txt

@@ -146,6 +146,8 @@ endif()
 
 # enable Edit and Continue in Visual Studio (for debugging)
 if(MSVC)
-  target_compile_options(${PROJECT_NAME}  INTERFACE "/ZI")
-  target_link_options(${PROJECT_NAME} INTERFACE "/INCREMENTAL")
+  if(CMAKE_BUILD_TYPE STREQUAL "Debug")
+    target_compile_options(${PROJECT_NAME}  INTERFACE "/ZI")
+    target_link_options(${PROJECT_NAME} INTERFACE "/INCREMENTAL")
+  endif()
 endif()

cmake/dependencies.cmake

@@ -8,7 +8,7 @@ include( shacl_FetchContent )
 
 shacl_FetchContent_Declare( tools
     GIT_REPOSITORY  ../../njoy/tools
-    GIT_TAG         902cf189f7a28012945577a9f77180a516e0e978  # tag: v0.4.3
+    GIT_TAG         22bae7709d012914f32c99c0fb7aab06f49c5f62  # tag: v0.4.3
     )
 
 #######################################################################

python/CMakeLists.txt

@@ -1,5 +1,6 @@
 if(ACEtk.python)
 
+    find_package( Python3 REQUIRED )
     pybind11_add_module( ACEtk.python
         ${CMAKE_CURRENT_SOURCE_DIR}/src/ACEtk.python.cpp
         ${CMAKE_CURRENT_SOURCE_DIR}/src/XsdirEntry.python.cpp