Skip to content

reef: cmake/arrow: don't treat warnings as errors#57375

Merged
batrick merged 1 commit intoceph:reeffrom
cbodley:wip-65885-reef
May 11, 2024
Merged

reef: cmake/arrow: don't treat warnings as errors#57375
batrick merged 1 commit intoceph:reeffrom
cbodley:wip-65885-reef

Conversation

@cbodley
Copy link
Contributor

@cbodley cbodley commented May 9, 2024

backport tracker: https://tracker.ceph.com/issues/65885


backport of #55303
parent tracker: https://tracker.ceph.com/issues/63130

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

with arrow's default BUILD_WARNING_LEVEL, -Werror is added to cflags and
debug builds fail on a warning about _FORTIFY_SOURCE

Fixes: https://tracker.ceph.com/issues/63130

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 5c1a57a)
@batrick
Copy link
Member

batrick commented May 9, 2024

Thank you @cbodley

@cbodley
Copy link
Contributor Author

cbodley commented May 9, 2024

@batrick
Copy link
Member

batrick commented May 9, 2024

jenkins test api

@cbodley
Copy link
Contributor Author

cbodley commented May 9, 2024

https://jenkins.ceph.com/job/ceph-windows-pull-requests/39921/

[2024-05-09T20:25:06.000Z] [isolated][googletest] ceph_test_libcephfs failed. Error: Command timed out (1800s): "cmd /c 'C:\ceph\ceph_test_libcephfs.exe --gtest_output=xml:C:\workspace\test_results\out\ceph_test_libcephfs\ceph_test_libcephfs_results.xml --gtest_filter="-LibCephFS.Deleg*" >> C:\workspace\test_results\out\ceph_test_libcephfs\ceph_test_libcephfs_results.log 2>&1'".

@cbodley
Copy link
Contributor Author

cbodley commented May 9, 2024

jenkins test windows

batrick added a commit to batrick/ceph that referenced this pull request May 10, 2024
* refs/pull/57375/head:
	cmake/arrow: don't treat warnings as errors
@batrick
Copy link
Member

batrick commented May 11, 2024

@batrick batrick merged commit a7db426 into ceph:reef May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants