Skip to content

Commit ec56b37

Browse files
authored
GH-48787: [C++] Disable -Werror for s2n-tls (#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. ### What changes are included in this PR? 1. Added -Wno-error specifically for the s2n target in cpp/cmake_modules/ThirdpartyToolchain.cmake. This allows the build to proceed despite intentional warnings in the s2n-tls project that Clang 18 now flags as errors. ### Are these changes tested? I am relying on the GitHub CI to verify that this suppression resolves the Ubuntu 24.04 compiler failures. ### Are there any user-facing changes? No. These changes only affect the C++ build system and third-party dependency management. * GitHub Issue: #48787 Authored-by: Meher <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
1 parent 1e8645e commit ec56b37

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3984,6 +3984,11 @@ function(build_awssdk)
39843984
# We don't need to link aws-lc. It's used only by s2n-tls.
39853985
elseif("${AWSSDK_PRODUCT}" STREQUAL "s2n-tls")
39863986
list(PREPEND AWSSDK_LINK_LIBRARIES s2n)
3987+
# Disable -Werror for s2n-tls: it has Clang 18 warnings that it intentionally allows.
3988+
# See: https://github.com/aws/s2n-tls/issues/5696
3989+
if(TARGET s2n)
3990+
target_compile_options(s2n PRIVATE -Wno-error)
3991+
endif()
39873992
else()
39883993
list(PREPEND AWSSDK_LINK_LIBRARIES ${AWSSDK_PRODUCT})
39893994
# This is for find_package(aws-*) in aws-crt-cpp and aws-sdk-cpp.

0 commit comments

Comments
 (0)