Add EXCLUDE_FROM_ALL for monero dependency (#221)
What changed, and why it matters
This is a one-line build-system change that tells CMake not to automatically build every target from the bundled Monero dependency when building monero-lws. It only affects which build targets are included by default and does not change any executable code, network behavior, or cryptographic logic.
No security action required; treat as a normal build-system improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit adds EXCLUDE_FROM_ALL to a FetchContent_Declare call for the monero dependency in CMakeLists.txt. This CMake option prevents targets defined in the fetched Monero project from being added to the default ‘all’ target, so only targets explicitly depended upon by monero-lws will be built. It is a build hygiene/efficiency change.
Changed components
CMakeLists.txt build configurationInspect captured patch +1 / −1
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 347d23e..5c89880 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -185,7 +185,7 @@ else () # NOT MONERO_BUILD_DIR
endif ()
include(FetchContent)
- FetchContent_Declare(monero SOURCE_DIR "${MONERO_SOURCE_DIR}")
+ FetchContent_Declare(monero SOURCE_DIR "${MONERO_SOURCE_DIR}" EXCLUDE_FROM_ALL)
if (NOT monero_POPULATED)
FetchContent_MakeAvailable(monero)
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.