mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-09-12 05:32:22 +02:00
Merge bitcoin/bitcoin#36045: test: avoid undersized Boost.Test signal stacks
9eba3aafa6test: avoid undersized Boost.Test signal stacks (Lőrinc) Pull request description: **Problem:** Boost.Test can fail while an Alpine CI test binary is starting, before any tests run. The required signal stack size depends on the runner's CPU features, while musl provides a fixed size. **Fix:** Use the [regular process stack](https://www.boost.org/doc/libs/latest/libs/test/doc/html/boost_test/utf_reference/link_references/config_disable_alt_stack.html) for Boost.Test signal handling in both unit-test binaries. Fixes #36026 ACKs for top commit: maflcko: lgtm ACK9eba3aafa6Tree-SHA512: 177a31ab325f4ebce0ad7e4679b171cd5a28787595bf0e117fc66731b8bf9157c4f5bef74cc6bb7651021bafb616fe5e9a25709abf30b7689d10ddd0fbd484cb
This commit is contained in:
@@ -7,6 +7,8 @@
|
||||
#include <util/byte_units.h>
|
||||
#include <util/fs.h>
|
||||
|
||||
// Boost.Test's SIGSTKSZ alternate stack can be smaller than Linux requires on musl.
|
||||
#define BOOST_TEST_DISABLE_ALT_STACK
|
||||
#define BOOST_TEST_MODULE Bitcoin Kernel Test Suite
|
||||
#include <boost/test/included/unit_test.hpp>
|
||||
|
||||
|
||||
@@ -5,6 +5,8 @@
|
||||
/**
|
||||
* See https://www.boost.org/doc/libs/1_78_0/libs/test/doc/html/boost_test/adv_scenarios/single_header_customizations/multiple_translation_units.html
|
||||
*/
|
||||
// Boost.Test's SIGSTKSZ alternate stack can be smaller than Linux requires on musl.
|
||||
#define BOOST_TEST_DISABLE_ALT_STACK
|
||||
#define BOOST_TEST_MODULE Bitcoin Core Test Suite
|
||||
|
||||
#include <boost/test/included/unit_test.hpp>
|
||||
|
||||
Reference in New Issue
Block a user