Resolves #1474
* Require CMake 3.12 at minimum to allow use of modern find_package
- c.f. https://cmake.org/cmake/help/v3.12/command/find_package.html
* Set range of valid CMake versions to allow for more modern CMakes
to be used if available.
- c.f. https://cliutils.gitlab.io/modern-cmake/chapters/basics.html
* Use FindPython which from CMake 3.12 which relies on find_package
advancements to select the components that are required. Select the
'Interpreter' and 'Development' components to find the required
executables and files.
- c.f. https://cmake.org/cmake/help/v3.12/module/FindPython.html
* Modern FindPython is capitalization sensative and will set the
following variables that are used throughout the project.
* `Python_FOUND`
* `Python_Interpreter_FOUND`
* `Python_EXECUTABLE`
* `Python_Development_FOUND`
- c.f. https://cmake.org/cmake/help/v3.12/module/FindPython.html#result-variables
You can view, comment on, or merge this pull request online at:
https://github.com/xrootd/xrootd/pull/1906
-- Commit Summary --
* feat: Set cmake_minimum_required to 3.12...3.25
* fix: Use CMake 3.12 FindPython to get Python components
-- File Changes --
M .github/workflows/build.yml (14)
M CMakeLists.txt (2)
M bindings/python/CMakeLists.txt (12)
M cmake/XRootDFindLibs.cmake (9)
M cmake/XRootDSummary.cmake (2)
M packaging/debian/rules (2)
M packaging/wheel/TestCXX14.txt (2)
-- Patch Links --
https://github.com/xrootd/xrootd/pull/1906.patch
https://github.com/xrootd/xrootd/pull/1906.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1906
You are receiving this because you are subscribed to this thread.
Message ID: <[log in to unmask]>
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
