Skip to content

Commit

Permalink
Include bitcoinconsensus.cpp directly in bitcoin_test
Browse files Browse the repository at this point in the history
This removes the library machinery which wasn't being tested anyway.
  • Loading branch information
theuni committed Dec 20, 2023
1 parent c8e80fc commit f30fb14
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions src/test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ generate_header_from_raw(data/asmap.raw)

add_executable(test_bitcoin
main.cpp
$<TARGET_OBJECTS:bitcoin_consensus>
../script/bitcoinconsensus.cpp
${CMAKE_CURRENT_BINARY_DIR}/data/asmap.raw.h
${CMAKE_CURRENT_BINARY_DIR}/data/base58_encode_decode.json.h
${CMAKE_CURRENT_BINARY_DIR}/data/bip341_wallet_vectors.json.h
Expand Down Expand Up @@ -143,7 +143,6 @@ target_link_libraries(test_bitcoin
bitcoin_cli
bitcoin_node
minisketch
$<TARGET_NAME_IF_EXISTS:bitcoinconsensus>
Boost::headers
)

Expand Down Expand Up @@ -174,8 +173,6 @@ if(ENABLE_WALLET)
endif()
endif()

make_bitcoinconsensus_dll_available(test_bitcoin)

install(TARGETS test_bitcoin
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
)

0 comments on commit f30fb14

Please sign in to comment.