From 10fd988a69e866c1ef36c8990b2046ccbf18920f Mon Sep 17 00:00:00 2001 From: William Woodall Date: Tue, 21 Oct 2025 15:16:39 -0700 Subject: [PATCH] Remove -Werror from Clang compile options While useful, in my opinion it's not appropriate to set `-Werror` the `CMakeLists.txt`, instead allowing the user to set it if they wish, or setting it during CI if you want to catch warnings during testing. Signed-off-by: William Woodall --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index b967f58..e6b23bb 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -78,7 +78,7 @@ if(BUILD_TESTING) ament_xmllint() if(CMAKE_CXX_COMPILER_ID MATCHES "Clang") - add_compile_options(-Wthread-safety -Werror) + add_compile_options(-Wthread-safety) endif() ament_add_gtest(test_asserts_ndebug test/test_asserts.cpp)