Skip to content

Commit

Permalink
Revert "[nrf fromtree] cmake: use the warnings_as_errors flag for cpp…
Browse files Browse the repository at this point in the history
… files"

Some nRF Connect SDK samples are generating warnings for C++ code.
nRF Connect SDK CI treats warnings as errors for code, but this was
previous not active for C++ source code.

Therfore revert the upstream warning-as-errors flag for C++ code.
Revert this commit when nRF Connect SDK C++ source code is clear from
warnings.

This reverts commit f3e3ced.

Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
  • Loading branch information
tejlmand authored and rlubos committed Oct 24, 2024
1 parent db1f0fc commit 97c23d6
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,6 @@ zephyr_compile_options($<$<COMPILE_LANGUAGE:CXX>:$<TARGET_PROPERTY:compiler-cpp,
# Extra warnings options for twister run
if (CONFIG_COMPILER_WARNINGS_AS_ERRORS)
zephyr_compile_options($<$<COMPILE_LANGUAGE:C>:$<TARGET_PROPERTY:compiler,warnings_as_errors>>)
zephyr_compile_options($<$<COMPILE_LANGUAGE:CXX>:$<TARGET_PROPERTY:compiler,warnings_as_errors>>)
zephyr_compile_options($<$<COMPILE_LANGUAGE:ASM>:$<TARGET_PROPERTY:asm,warnings_as_errors>>)
zephyr_link_libraries($<TARGET_PROPERTY:linker,warnings_as_errors>)
endif()
Expand Down

0 comments on commit 97c23d6

Please sign in to comment.