Skip to content

Commit

Permalink
fix: Avoid targeting honk test files when testing is disabled (AztecP…
Browse files Browse the repository at this point in the history
  • Loading branch information
phated authored Feb 9, 2023
1 parent 727f464 commit c155f79
Showing 1 changed file with 18 additions and 16 deletions.
34 changes: 18 additions & 16 deletions barretenberg/cpp/src/aztec/honk/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,18 +1,20 @@
barretenberg_module(honk numeric ecc srs proof_system transcript)

# TODO: Re-enable all these warnings once PoC is finished
if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
target_compile_options(
honk_test_objects
PRIVATE
-Wno-error=unused-variable
)
elseif(CMAKE_CXX_COMPILER_ID MATCHES "GNU")
target_compile_options(
honk_test_objects
PRIVATE
-Wno-error=maybe-uninitialized
-Wno-error=uninitialized
-Wno-error=unused-variable
)
endif()
if(TESTING)
# TODO: Re-enable all these warnings once PoC is finished
if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
target_compile_options(
honk_test_objects
PRIVATE
-Wno-error=unused-variable
)
elseif(CMAKE_CXX_COMPILER_ID MATCHES "GNU")
target_compile_options(
honk_test_objects
PRIVATE
-Wno-error=maybe-uninitialized
-Wno-error=uninitialized
-Wno-error=unused-variable
)
endif()
endif()

0 comments on commit c155f79

Please sign in to comment.