Skip to content

Commit

Permalink
Revert "SWPROT-8953: libs2: Relax Werror for release, to be reverted"
Browse files Browse the repository at this point in the history
One would argue that libs2 warnings should not be addressed
in the consumming project, but until a better architecture is in place,
we can enable it to detect issues and help fixing them at the root.
If for any reason having this patch is a problem,
I encourage to cherry-pick the 1st Bypass change
(that would be applied in next release)

This reverts commit 436ef26.

Origin: #37
Relate-to: #28
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
  • Loading branch information
rzr committed Feb 18, 2025
1 parent 14a1b28 commit 6854282
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
@@ -8,7 +8,7 @@ if("${PROJECT_NAME}" STREQUAL "")
add_definitions(-Dx86)

if(CMAKE_COMPILER_IS_GNUCC OR CMAKE_C_COMPILER_ID MATCHES "Clang")
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall")
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall -Werror")
# For gov/cobertura: do not use .c.o on the object files, only .o
set(CMAKE_C_OUTPUT_EXTENSION_REPLACE 1)
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fprofile-arcs -ftest-coverage")

0 comments on commit 6854282

Please sign in to comment.