signals: remove boost includes where possible
What the developer wrote
Authored by Cory Fields
signals: remove boost includes where possible
In both of these cases, boost was only needed for signals and not for
multi_index/test.
What changed, and why it matters
This is a routine build cleanup. A developer removed two leftover references to the Boost C++ library from the project's build instructions because those parts of the code no longer use Boost. There is no security issue here.
No security action needed. This is a normal maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies src/CMakeLists.txt to remove Boost::headers from the link dependencies of bitcoin_common and bitcoin-wallet targets. The commit message states that Boost was only needed for signals in these cases and is no longer required. This is a dependency cleanup with no functional code changes.
Changed components
src/CMakeLists.txtInspect captured patch +0 / −2
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index cf1f26c9..4eb7ae84 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -143,7 +143,6 @@ target_link_libraries(bitcoin_common
bitcoin_util
univalue
secp256k1
- Boost::headers
$<TARGET_NAME_IF_EXISTS:USDT::headers>
$<$<PLATFORM_ID:Windows>:ws2_32>
)
@@ -166,7 +165,6 @@ if(ENABLE_WALLET)
bitcoin_wallet
bitcoin_common
bitcoin_util
- Boost::headers
)
install_binary_component(bitcoin-wallet HAS_MANPAGE)
endif()
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.