New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: Remove unsafe BOOST_TEST_MESSAGE #18472
Conversation
cc @mzumsande |
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
fa93702
to
fa3cc0b
Compare
tested ACK fa3cc0b Thank you! |
Tested ACK fa3cc0b Maybe test/README.md could mention the DEBUG_LOG_OUT option which I find really helpful, and possibly also a warning that BOOST_TEST_MESSAGE, which is mentioned there, is not threadsafe. |
Thanks, I am going to go ahead and merge this bug fix. Will update the docs in a new commit. |
Done here: #18486 |
fa3cc0b test: Remove unsafe BOOST_TEST_MESSAGE (MarcoFalke) Pull request description: Fixes bitcoin#17987 Can be tested with ``` ./src/test/test_bitcoin -l test_suite -- DEBUG_LOG_OUT ACKs for top commit: fjahr: tested ACK fa3cc0b mzumsande: Tested ACK fa3cc0b Tree-SHA512: f63b110d77882cd7c0d7574ff6c9c948db8febb3400ecdac45164746b587b0fa223463041801271b3959267ddc1d9a4a67ba76939e242e7dd2f92a2834a400a0
7777703 doc: Explain new test logging (MarcoFalke) Pull request description: Explain logging added in bitcoin#18472 and bitcoin#16975 ACKs for top commit: jonatack: ACK 7777703 Tree-SHA512: 3a0aa7bab32a6753d8894d29cf82604b044b23e512102dd275b717eefda3c2212dbf43ea7e9155267350dd9f3bc5badba2eb660152db3efeab30a04f52126c95
Fixes #17987
Can be tested with