Skip to content

Commit

Permalink
Treat warnings as errors only in CI tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
vkucera committed Apr 30, 2024
1 parent 99f1307 commit 15994c1
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions dependencies/O2PhysicsCompileFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,11 @@

include_guard()

# Set warnings for both gcc and clang.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wno-error \
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wno-error")
# Treat warnings as errors in CI tests.
if(DEFINED ENV{ALIBUILD_O2PHYSICS_TESTS})
# Set warnings for both gcc and clang.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
-Werror=NSObject-attribute \
-Werror=address \
-Werror=address-of-packed-member \
Expand Down Expand Up @@ -92,9 +95,9 @@ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wno-error \
-Werror=varargs \
-Werror=vexing-parse \
")
# Add warnings for clang only.
if(APPLE)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
# Add warnings for clang only.
if(APPLE)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
-Werror=#pragma-messages \
-Werror=#warnings \
-Werror=CFString-literal \
Expand Down Expand Up @@ -638,9 +641,9 @@ if(APPLE)
-Werror=write-strings \
-Werror=xor-used-as-pow \
")
# Add warnings for gcc only.
elseif(UNIX)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
# Add warnings for gcc only.
elseif(UNIX)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
-Werror=aggressive-loop-optimizations \
-Werror=analyzer-double-fclose \
-Werror=analyzer-double-free \
Expand Down Expand Up @@ -733,6 +736,7 @@ elseif(UNIX)
-Werror=volatile-register-var \
-Werror=zero-length-bounds \
")
endif()
endif()

IF (ENABLE_TIMETRACE)
Expand Down

0 comments on commit 15994c1

Please sign in to comment.