GH-48787: [C++] Disable `-Werror` for s2n-tls by meher745 · Pull Request #48791 · apache/arrow

meher745 · 2026-01-08T17:52:41Z

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?

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: [CI][C++] test-ubuntu-24.04-cpp-thread-sanitizer fails to build s2n-tls #48787

github-actions · 2026-01-09T00:52:05Z

⚠️ GitHub issue #48787 has been automatically assigned in GitHub to PR creator.

kou · 2026-01-09T00:53:01Z

@github-actions crossbow submit test-ubuntu-24.04-cpp-thread-sanitizer

github-actions · 2026-01-09T00:53:07Z

⚠️ GitHub issue #48787 has been automatically assigned in GitHub to PR creator.

github-actions · 2026-01-09T00:55:11Z

Revision: 7ebcda0

Submitted crossbow builds: ursacomputing/crossbow @ actions-82a9f13dfd

Task	Status
test-ubuntu-24.04-cpp-thread-sanitizer

kou · 2026-01-09T01:23:43Z

Can we disable -Werror for bundled s2n-tls?

HyukjinKwon · 2026-01-09T01:36:43Z

I have some changes in #48797 could be applied here if that looks good!

meher745 · 2026-01-09T04:06:54Z

I have some changes in #48797 could be applied here if that looks good!

Thank you so much! @HyukjinKwon

kou · 2026-01-09T04:28:24Z

cpp/CMakeLists.txt

+# Allow older bundled dependencies to build with CMake 4.0+
+set(CMAKE_POLICY_VERSION_MINIMUM 3.5)


Do we need this?

While building on my local system, I was getting this error. So I thought it would be better to comply with this requirement.

Just my two cents. I would rather make this PR minimized to fix the issue, and separately file an issue for this and make a PR :-).

Yes. We should not mix separated problems as much as possible.

Let's revert this change from this PR, open a separated issue and let's discuss it there.

FYI: We already have similar code:

arrow/cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines 1023 to 1026 in b126c6e

# We set CMAKE_POLICY_VERSION_MINIMUM temporarily due to failures with CMake 4

# We should remove it once we have updated the dependencies:

# https://github.com/apache/arrow/issues/45985

set(CMAKE_POLICY_VERSION_MINIMUM 3.5)

kou · 2026-01-09T04:28:40Z

@github-actions crossbow submit test-ubuntu-24.04-cpp-thread-sanitizer

github-actions · 2026-01-09T04:30:47Z

Revision: 5852bc1

Submitted crossbow builds: ursacomputing/crossbow @ actions-d8e1dd6f27

Task	Status
test-ubuntu-24.04-cpp-thread-sanitizer

meher745 · 2026-01-09T05:42:35Z

I have reverted the extra changes as requested. This PR now only contains the -Wno-error suppression.

I will open a separate PR shortly for the CMake 4.0 compatibility fix. Thank you once again!

HyukjinKwon · 2026-01-09T05:45:33Z

The change LGTM! BTW seems like there was testing deletion that mistakenly happened

Might need to restore 👍

meher745 · 2026-01-09T06:00:11Z

The change LGTM! BTW seems like there was testing deletion that mistakenly happened
Might need to restore 👍

Done! Thank you!

kou · 2026-01-09T06:20:57Z

@github-actions crossbow submit test-ubuntu-24.04-cpp-thread-sanitizer

kou · 2026-01-09T06:21:04Z

Could you update the PR description?

github-actions · 2026-01-09T06:23:25Z

Revision: 92b2cd0

Submitted crossbow builds: ursacomputing/crossbow @ actions-0a32f5ea59

Task	Status
test-ubuntu-24.04-cpp-thread-sanitizer

kou · 2026-01-09T06:23:29Z

I will open a separate PR shortly for the CMake 4.0 compatibility fix.

Could you open an issue before this? It's for "openness" of the Apache way.
See also: https://theapacheway.com/open/

And could you add a "take" comment to #48787 to assign the issue to you?

raulcd · 2026-01-09T14:23:57Z

@github-actions crossbow submit -g cpp

raulcd · 2026-01-09T14:24:47Z

The Python / AMD64 Conda Python 3.10 Sphinx & Numpydoc failure is unrelated and has been fixed on main now.

github-actions · 2026-01-09T14:26:36Z

Revision: 92b2cd0

Submitted crossbow builds: ursacomputing/crossbow @ actions-5488c7cb94

Task	Status
example-cpp-minimal-build-static
example-cpp-minimal-build-static-system-dependency
example-cpp-tutorial
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp-ubuntu-22.04-cuda-11.7.1
test-cuda-cpp-ubuntu-24.04-cuda-13.0.2
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-debian-experimental-cpp-gcc-15
test-fedora-42-cpp
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-bundled
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-bundled-offline
test-ubuntu-24.04-cpp-gcc-13-bundled
test-ubuntu-24.04-cpp-gcc-14
test-ubuntu-24.04-cpp-minimal-with-formats
test-ubuntu-24.04-cpp-thread-sanitizer

raulcd

LGTM

cpp/cmake_modules/ThirdpartyToolchain.cmake

kou

+1

conbench-apache-arrow · 2026-01-11T15:33:17Z

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit ec56b37.

There weren't enough matching historic benchmark results to make a call on whether there were regressions.

The full Conbench report has more details.

build: upgrade s2n-tls and fix CMake 4.0 compatibility

7ebcda0

github-actions bot added Component: C++ awaiting review Awaiting review labels Jan 8, 2026

kou changed the title ~~MINOR: [Cpp] Upgrade s2n-tls to v1.6.4 and fix CMake 4.0 compatibility~~ GH-48787: [C++] Upgrade s2n-tls to v1.6.4 and fix CMake 4.0 compatibility Jan 9, 2026

build: disable -Werror for s2n-tls as recommended

5852bc1

kou reviewed Jan 9, 2026

View reviewed changes

github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Jan 9, 2026

build: disable -Werror for s2n-tls to fix Ubuntu 24.04 CI

0b3cc89

github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Jan 9, 2026

chore: restore testing submodule to match upstream

92b2cd0

raulcd reviewed Jan 9, 2026

View reviewed changes

cpp/cmake_modules/ThirdpartyToolchain.cmake Show resolved Hide resolved

github-actions bot added awaiting changes Awaiting changes and removed awaiting change review Awaiting change review labels Jan 9, 2026

kou changed the title ~~GH-48787: [C++] Upgrade s2n-tls to v1.6.4 and fix CMake 4.0 compatibility~~ GH-48787: [C++] Disable -Werror for s2n-tls Jan 9, 2026

kou approved these changes Jan 9, 2026

View reviewed changes

raulcd approved these changes Jan 10, 2026

View reviewed changes

github-actions bot added awaiting merge Awaiting merge and removed awaiting changes Awaiting changes labels Jan 10, 2026

kou merged commit ec56b37 into apache:main Jan 11, 2026
51 of 52 checks passed

kou removed the awaiting merge Awaiting merge label Jan 11, 2026

kou mentioned this pull request Jan 11, 2026

[CI][C++] test-ubuntu-24.04-cpp-thread-sanitizer fails to build s2n-tls #48787

Closed

pitrou added the backport-candidate label Jan 29, 2026

raulcd removed the backport-candidate label Feb 3, 2026

		# Allow older bundled dependencies to build with CMake 4.0+
		set(CMAKE_POLICY_VERSION_MINIMUM 3.5)

	# We set CMAKE_POLICY_VERSION_MINIMUM temporarily due to failures with CMake 4
	# We should remove it once we have updated the dependencies:
	# https://github.com/apache/arrow/issues/45985
	set(CMAKE_POLICY_VERSION_MINIMUM 3.5)

Conversation

meher745 commented Jan 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

kou commented Jan 9, 2026

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

kou commented Jan 9, 2026

Uh oh!

HyukjinKwon commented Jan 9, 2026

Uh oh!

meher745 commented Jan 9, 2026

Uh oh!

kou Jan 9, 2026

Choose a reason for hiding this comment

Uh oh!

meher745 Jan 9, 2026

Choose a reason for hiding this comment

Uh oh!

HyukjinKwon Jan 9, 2026

Choose a reason for hiding this comment

Uh oh!

kou Jan 9, 2026

Choose a reason for hiding this comment

Uh oh!

kou commented Jan 9, 2026

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

meher745 commented Jan 9, 2026

Uh oh!

HyukjinKwon commented Jan 9, 2026

Uh oh!

meher745 commented Jan 9, 2026

Uh oh!

kou commented Jan 9, 2026

Uh oh!

kou commented Jan 9, 2026

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

kou commented Jan 9, 2026

Uh oh!

raulcd commented Jan 9, 2026

Uh oh!

raulcd commented Jan 9, 2026

Uh oh!

github-actions bot commented Jan 9, 2026

Uh oh!

raulcd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

kou left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

conbench-apache-arrow bot commented Jan 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

meher745 commented Jan 8, 2026 •

edited

Loading