Add documentation for the PYTHON_MODULE_EXTENSION and only set it when crosscompiling to allow Python version upgrades.

AlexisTM · AlexisTM · commit b1862e2c922c · 2022-10-19T12:23:18.000Z
Signed-off-by: Alexis Paques &lt;paa1ti@bosch.com&gt;
diff --git a/cmake/Modules/FindPythonExtra.cmake b/cmake/Modules/FindPythonExtra.cmake
@@ -26,6 +26,8 @@
 # - PythonExtra_INCLUDE_DIRS: The paths to the directories where the Python
 #    headers are installed.
 # - PythonExtra_LIBRARIES: The paths to the Python libraries.
+# - PYTHON_MODULE_EXTENSION: The full module extension, as the suffix+extension.
+#    This is required for packages using pybind11 when crosscompiling.
 #
 # Example usage:
 #
@@ -219,3 +221,7 @@ find_package_handle_standard_args(PythonExtra
   FOUND_VAR PythonExtra_FOUND
   REQUIRED_VARS ${_required_vars}
 )
+
+if(DEFINED CMAKE_SYSROOT)
+  set(PYTHON_MODULE_EXTENSION ${PythonExtra_EXTENSION_SUFFIX}${PythonExtra_EXTENSION_EXTENSION})
+endif()

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@`
`26`	`26`	`# - PythonExtra_INCLUDE_DIRS: The paths to the directories where the Python`
`27`	`27`	`# headers are installed.`
`28`	`28`	`# - PythonExtra_LIBRARIES: The paths to the Python libraries.`
	`29`	`+# - PYTHON_MODULE_EXTENSION: The full module extension, as the suffix+extension.`
	`30`	`+# This is required for packages using pybind11 when crosscompiling.`
`29`	`31`	`#`
`30`	`32`	`# Example usage:`
`31`	`33`	`#`
`@@ -219,3 +221,7 @@ find_package_handle_standard_args(PythonExtra`
`219`	`221`	`FOUND_VAR PythonExtra_FOUND`
`220`	`222`	`REQUIRED_VARS ${_required_vars}`
`221`	`223`	`)`
	`224`	`+`
	`225`	`+if(DEFINED CMAKE_SYSROOT)`
	`226`	`+ set(PYTHON_MODULE_EXTENSION ${PythonExtra_EXTENSION_SUFFIX}${PythonExtra_EXTENSION_EXTENSION})`
	`227`	`+endif()`