Skip to content

cmake: Fix test/config.ini generation#109

Merged
hebasto merged 2 commits into
cmake-stagingfrom
240302-cmake-CA
Mar 3, 2024
Merged

cmake: Fix test/config.ini generation#109
hebasto merged 2 commits into
cmake-stagingfrom
240302-cmake-CA

Conversation

@hebasto
Copy link
Copy Markdown
Owner

@hebasto hebasto commented Mar 2, 2024

I've noticed this bug during extensive testing in different scenarios after the recent sync.

Should be trivial to review and test by observing the content of the generated test/config.ini file in the build directory.

@hebasto hebasto added the bug Something isn't working label Mar 2, 2024
Comment thread test/CMakeLists.txt
set_configure_variable(WITH_ZMQ ENABLE_ZMQ)
set_configure_variable(ENABLE_EXTERNAL_SIGNER ENABLE_EXTERNAL_SIGNER)
set_configure_variable(ENABLE_TRACING ENABLE_USDT_TRACEPOINTS)
set_configure_variable(WITH_EXTERNAL_SIGNER ENABLE_EXTERNAL_SIGNER)
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #79.

@hebasto
Copy link
Copy Markdown
Owner Author

hebasto commented Mar 2, 2024

Copy link
Copy Markdown

@pablomartin4btc pablomartin4btc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

crACK & tACK 94ea7fd

cat build/test/config.ini

...
[components]
# Which components are enabled. These are commented out by `configure` if they were disabled when running config.
ENABLE_WALLET=true
USE_SQLITE=true
#USE_BDB=true
ENABLE_CLI=true
ENABLE_BITCOIN_UTIL=true
ENABLE_WALLET_TOOL=true
ENABLE_BITCOIND=true
ENABLE_FUZZ_BINARY=true
ENABLE_ZMQ=true
#ENABLE_EXTERNAL_SIGNER=true
ENABLE_USDT_TRACEPOINTS=true

@hebasto hebasto merged commit d965822 into cmake-staging Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants