meher745 opened a new pull request, #48791:
URL: https://github.com/apache/arrow/pull/48791

   ### Rationale for this change
   
   Fixes the test-ubuntu-24.04-cpp-thread-sanitizer failure where s2n-tls 
builds were failing due to strict compiler warnings (documentation and integer 
precision) on newer Clang versions. It also resolves a configuration blocker 
for developers using CMake 4.0+.
   
   ### What changes are included in this PR?
   
   1. Upgrade s2n-tls: Bumped version from v1.5.23 to v1.6.4 in 
cpp/thirdparty/versions.txt (including checksum update).
   
   2. CMake Policy Fix: Added set(CMAKE_POLICY_VERSION_MINIMUM 3.5) to 
cpp/CMakeLists.txt to allow modern CMake versions to build legacy bundled 
dependencies like RapidJSON.
   
   ### Are these changes tested?
   
   I verified that the CMake policy fix resolves the configuration-stage errors 
(specifically for RapidJSON) when using CMake 4.2.1. I was unable to build the 
s2n-tls target locally due to environment mismatches on macOS arm64, so I am 
relying on the GitHub CI to verify that the version bump resolves the Ubuntu 
24.04 compiler failures. If not, I'll look into the intricacies in a better 
way. 
   
   ### Are there any user-facing changes?
   
   No. These changes only affect the C++ build system and third-party 
dependency management.
   
   GitHub Issue: #48787
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to