diff options
author | merge-script <fanquake@gmail.com> | 2024-09-03 12:15:28 +0100 |
---|---|---|
committer | merge-script <fanquake@gmail.com> | 2024-09-03 12:15:28 +0100 |
commit | 9cb9651d92ddb5d92724f6a52440601c7a0bbcf8 (patch) | |
tree | 1736e4bf7dd0e7c14fe2b22853d0ee926825a347 /src/test/validation_chainstatemanager_tests.cpp | |
parent | 4c526f575cde43cc49c0236f4d2b15a5fbb6d7ab (diff) | |
parent | 787dfaf084a3952319778da9cbcda9d7d619e4ee (diff) |
Merge bitcoin/bitcoin#30778: build: Fix linking for `fuzz` target when building with MSan
787dfaf084a3952319778da9cbcda9d7d619e4ee ci: Do not override `-g -O1` set in `MSAN_FLAGS` (Hennadii Stepanov)
26c460aa8b5decfd08d931b9b3f80be5c13c7528 build: Fix linking for `fuzz` target when building with MSan (Hennadii Stepanov)
Pull request description:
The first commit fixes https://github.com/bitcoin/bitcoin/issues/30760.
The second commit:
1. Preserves `-g -O1` set in `MSAN_FLAGS`. Since configuration-specific flags override general flags, these are set to empty strings. A similar approach is used in the OSS-Fuzz repository.
2. Sets the "Debug" build configuration when depends are built with `DEBUG=1`, ensuring that `linux_debug_CPPFLAGS` from depends are passed to the main build system.
ACKs for top commit:
maflcko:
review-only ACK 787dfaf084a3952319778da9cbcda9d7d619e4ee
fanquake:
ACK 787dfaf084a3952319778da9cbcda9d7d619e4ee - as a follow up it would be good to:
Tree-SHA512: c324390d1dbda30f82025d8482ddb0cfa0395f9ba225a2ddce05a123c65e0622a6a1d5f0fa03f09e21d62792431cf3da5c49e41a3ac7f7a958d0392a0430f29c
Diffstat (limited to 'src/test/validation_chainstatemanager_tests.cpp')
0 files changed, 0 insertions, 0 deletions