What the developer wrote
Authored by Hennadii Stepanov
cmake: Fix target name
The executable target run by ctest is `mptest`.
This change removes unnecessary steps when building the `codegen`
target.
What changed, and why it matters
This is a one-character CMake build-system fix. The developer corrected the name of an internal test target from 'mptests' to 'mptest' so that automated testing (ctest) can find and run it. It has no effect on the Bitcoin network, wallets, consensus rules, or user-facing security.
No security action needed. Treat as a normal build/test maintenance fix.
Security signals we found
No security-relevant code change
Build-system-only change
Corrects a target name mismatch in CMake configuration
Evidence from the diff
In cmake/libmultiprocess.cmake, set_target_properties was referencing a non-existent target ‘mptests’ instead of the actual executable target ‘mptest’. The patch changes that single identifier. The only consequence of the bug was that the multiprocess test target remained excluded from the default ‘all’ build target, so ctest would not run those tests automatically. The patch restores intended CI/test behavior.
Changed components
cmake/libmultiprocess.cmakelibmultiprocess test target registrationInspect captured patch +1 / −1
diff --git a/cmake/libmultiprocess.cmake b/cmake/libmultiprocess.cmake
index 5db8f4e9..32da8f8c 100644
--- a/cmake/libmultiprocess.cmake
+++ b/cmake/libmultiprocess.cmake
@@ -27,7 +27,7 @@ function(add_libmultiprocess subdir)
mark_as_advanced(CapnProto_kj-tls_IMPORTED_LOCATION)
if(BUILD_TESTS)
# Add tests to "all" target so ctest can run them
- set_target_properties(mptests PROPERTIES EXCLUDE_FROM_ALL OFF)
+ set_target_properties(mptest PROPERTIES EXCLUDE_FROM_ALL OFF)
endif()
# Exclude examples from compilation database, because the examples are not
# built by default, and they contain generated c++ code. Without this
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.