CMake: Switch to using find_package(Python3) and Python3_EXECUTABLE

The old style used was deprecated in CMake 3.12, so this should not negatively impact our compatibility (requires 3.16)
KhronosGroup · Aug 29, 2024 · 1d4be37 · 1d4be37
1 parent 4b9834d
commit 1d4be37
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -21,7 +21,7 @@
 cmake_minimum_required(VERSION 3.16)
 project(OPENXR)
 
-find_package(PythonInterp 3)
+find_package(Python3)
 include(CTest)
 
 # Enable IDE GUI folders.  "Helper targets" that don't have interesting source code should set their FOLDER property to this
@@ -44,7 +44,7 @@ option(
     OFF
 )
 
-if(BUILD_FORCE_GENERATION AND NOT PYTHON_EXECUTABLE)
+if(BUILD_FORCE_GENERATION AND NOT Python3_EXECUTABLE)
     message(FATAL_ERROR "BUILD_FORCE_GENERATION requires Python")
 endif()
 

diff --git a/include/openxr/CMakeLists.txt b/include/openxr/CMakeLists.txt
@@ -79,7 +79,7 @@ else()
             OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${output}"
             COMMAND
                 "${CMAKE_COMMAND}" -E env "PYTHONPATH=${CODEGEN_PYTHON_PATH}"
-                "${PYTHON_EXECUTABLE}"
+                "${Python3_EXECUTABLE}"
                 "${PROJECT_SOURCE_DIR}/specification/scripts/genxr.py" -registry
                 "${PROJECT_SOURCE_DIR}/specification/registry/xr.xml" -o
                 "${CMAKE_CURRENT_BINARY_DIR}" ${output}

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -429,7 +429,7 @@ macro(run_xr_xml_generate dependency output)
         )
         list(APPEND GENERATED_OUTPUT "${CMAKE_CURRENT_SOURCE_DIR}/${output}")
     else()
-        if(NOT PYTHON_EXECUTABLE)
+        if(NOT Python3_EXECUTABLE)
             message(
                 FATAL_ERROR
                     "Python 3 not found, but pre-generated ${CMAKE_CURRENT_SOURCE_DIR}/${output} not found"
@@ -439,7 +439,7 @@ macro(run_xr_xml_generate dependency output)
             OUTPUT "${output}"
             COMMAND
                 "${CMAKE_COMMAND}" -E env "PYTHONPATH=${CODEGEN_PYTHON_PATH}"
-                "${PYTHON_EXECUTABLE}"
+                "${Python3_EXECUTABLE}"
                 "${PROJECT_SOURCE_DIR}/src/scripts/src_genxr.py" -registry
                 "${PROJECT_SOURCE_DIR}/specification/registry/xr.xml"
                 "${output}"
@@ -453,7 +453,7 @@ macro(run_xr_xml_generate dependency output)
                 ${ARGN}
             VERBATIM
             COMMENT
-                "Generating ${output} using ${PYTHON_EXECUTABLE} on ${dependency}"
+                "Generating ${output} using ${Python3_EXECUTABLE} on ${dependency}"
         )
         set_source_files_properties(${output} PROPERTIES GENERATED TRUE)
         list(APPEND GENERATED_OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${output}")
@@ -475,7 +475,7 @@ macro(
         OUTPUT "${filename}"
         COMMAND
             "${CMAKE_COMMAND}" -E env "PYTHONPATH=${CODEGEN_PYTHON_PATH}"
-            "${PYTHON_EXECUTABLE}"
+            "${Python3_EXECUTABLE}"
             "${PROJECT_SOURCE_DIR}/src/scripts/generate_api_layer_manifest.py"
             -f "${filename}" -n ${layername} -l ${libfile} -a ${MAJOR}.${MINOR}
             -v ${version} ${genbad} -d ${desc}

diff --git a/src/tests/loader_test/test_runtimes/CMakeLists.txt b/src/tests/loader_test/test_runtimes/CMakeLists.txt
@@ -49,7 +49,7 @@ macro(gen_xr_runtime_json filename libfile)
     add_custom_command(
         OUTPUT ${filename}
         COMMAND
-            "${PYTHON_EXECUTABLE}"
+            "${Python3_EXECUTABLE}"
             "${PROJECT_SOURCE_DIR}/src/scripts/generate_runtime_manifest.py" -f
             ${filename} -l ${libfile} ${ARGN}
         DEPENDS "${PROJECT_SOURCE_DIR}/src/scripts/generate_runtime_manifest.py"