What the developer wrote
Authored by tobtoht
fcmp_pp_rust: fix docker build
What changed, and why it matters
This is a build-system patch that lets the Rust part of Monero compile correctly when targeting Linux systems that use the lightweight 'musl' C library (common in Docker containers). It adds a check so the build picks the right Rust toolchain variant. There is no indication this changes any runtime behavior, cryptographic logic, or network code.
No security action needed; treat as ordinary build fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies src/fcmp_pp/fcmp_pp_rust/CMakeLists.txt to detect CMAKE_CXX_COMPILER_TARGET containing ‘musl’ and set RUST_TOOLCHAIN to ‘-musl’. Previously the toolchain selection only handled glibc-based armv7. This is a build-configuration fix for musl-based Docker/toolchain targets; it does not alter fcmp_pp cryptographic code, consensus rules, or runtime security boundaries.
Changed components
src/fcmp_pp/fcmp_pp_rust/CMakeLists.txtInspect captured patch +4 / −1
diff --git a/src/fcmp_pp/fcmp_pp_rust/CMakeLists.txt b/src/fcmp_pp/fcmp_pp_rust/CMakeLists.txt
index c63b61b..2aad35f 100644
--- a/src/fcmp_pp/fcmp_pp_rust/CMakeLists.txt
+++ b/src/fcmp_pp/fcmp_pp_rust/CMakeLists.txt
@@ -80,7 +80,10 @@ elseif(CMAKE_SYSTEM_NAME MATCHES "(SunOS|Solaris)")
endif()
else()
set(RUST_PLATFORM "unknown-linux")
- if(RUST_ARCH MATCHES "armv7")
+
+ if(CMAKE_CXX_COMPILER_TARGET MATCHES "musl")
+ set(RUST_TOOLCHAIN "-musl")
+ elseif(RUST_ARCH MATCHES "armv7")
# Rust does support non-HF, yet Monero assumes HF for armv7
set(RUST_TOOLCHAIN "-gnueabihf")
else()
Why this scored 16/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.