This is a rewrite of the build system of the Python bindings, to solve some of the open tickets we have.

@henryiii Could you please review these changes? I'd appreciate your input.

@adriansev Could you also have a look to check that it would work for your use cases? Some of the open tickets this solves were filed by you. Thank you!

You can view, comment on, or merge this pull request online at:

https://github.com/xrootd/xrootd/pull/2025

Commit Summary

79c966a [CMake] Use an interface library for bundled isa-l
b8741c1 [CMake] Replace setting blank link properties with private linking
fcd4910 [CMake] Replace include_directories with target_include_directories
43a59b8 [CI] Use DEBUG log level during CMake configuration
67a83e3 [RPM] Update makesrpm.sh to not use genversion.sh
66d7514 [RPM] Update compat version in spec file
2d3a8da Use CMake to detect/set XRootD version and generate XrdVersion.hh
4704126 [CI] Enable devtoolset-7 on CentOS 7
8dc208e [CMake] Enforce C++14 being available
ac2a150 [Python] Modernize build system
6ff2de3 Revert "[CI] Do not update pip, setuptools, and wheel for sdist build"

File Changes

(64 files)

M .gitattributes (2)
M .github/workflows/build.yml (13)
M .gitignore (1)
M CMakeLists.txt (25)
R MANIFEST.in (6)
A VERSION (1)
D VERSION_INFO (1)
D bindings/python/.gitattributes (1)
M bindings/python/.gitignore (1)
M bindings/python/CMakeLists.txt (114)
M bindings/python/MANIFEST.in (3)
A bindings/python/README (6)
D bindings/python/README.rst (16)
A bindings/python/VERSION (1)
A bindings/python/project.toml (2)
A bindings/python/setup.py (143)
D bindings/python/setup.py.in (126)
A bindings/python/src/CMakeLists.txt (49)
M cmake/XRootDDefaults.cmake (2)
M cmake/XRootDOSDefs.cmake (1)
A cmake/XRootDVersion.cmake (77)
M docker/xrd-docker (15)
M genversion.sh (291)
M packaging/debian/rules (6)
M packaging/makesrpm.sh (71)
M packaging/rhel/xrootd.spec.in (9)
D packaging/wheel/TestCXX14.txt (95)
D packaging/wheel/has_c++14.sh (12)
D packaging/wheel/install.sh (61)
M packaging/wheel/publish.sh (8)
D packaging/wheel/setup.py (220)
A project.toml (2)
A setup.py (116)
M src/CMakeLists.txt (4)
M src/XrdApps.cmake (17)
D src/XrdCeph/genversion.sh (239)
M src/XrdCeph/src/XrdCeph.cmake (19)
M src/XrdCl/CMakeLists.txt (6)
M src/XrdClHttp/CMakeLists.txt (2)
M src/XrdCrypto.cmake (17)
M src/XrdEc/CMakeLists.txt (6)
M src/XrdFfs.cmake (5)
M src/XrdHttp.cmake (3)
M src/XrdIsal.cmake (14)
M src/XrdMacaroons.cmake (13)
M src/XrdOssCsi.cmake (7)
M src/XrdPfc.cmake (14)
M src/XrdPlugins.cmake (56)
M src/XrdPosix.cmake (10)
M src/XrdSciTokens.cmake (15)
M src/XrdSec.cmake (35)
M src/XrdSecgsi.cmake (21)
M src/XrdSeckrb5.cmake (9)
M src/XrdSecztn.cmake (9)
M src/XrdServer.cmake (12)
M src/XrdSsi.cmake (22)
M src/XrdTpc.cmake (7)
M src/XrdUtils.cmake (7)
M src/XrdVersion.hh.in (7)
M src/XrdVoms.cmake (9)
M src/XrdXml.cmake (5)
M tests/XrdCephTests/CMakeLists.txt (4)
M tests/XrdClTests/CMakeLists.txt (6)
M tests/XrdEcTests/CMakeLists.txt (9)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/2025", "url": "https://github.com/xrootd/xrootd/pull/2025", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

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