What the developer wrote
Authored by k9ert
fix: macos build (#325)
What changed, and why it matters
This commit adjusts compiler warning flags in the build configuration so that the project compiles on macOS. On Linux, two specific warning suppressions remain enabled; on non-Linux systems (such as macOS), those suppressions are removed because the underlying compiler warnings do not exist or behave differently. There is no security-relevant change to the firmware or application logic.
No security action required. Treat as a routine build-system fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The Makefile change conditionally sets MPY_CFLAGS. If uname reports Linux, it keeps -Wno-dangling-pointer -Wno-enum-int-mismatch; otherwise it leaves the variable empty. This is a portability/build fix for macOS and does not alter runtime behavior, attack surface, or binary output in a security-relevant way.
Changed components
Makefile build configurationInspect captured patch +5 / −1
diff --git a/Makefile b/Makefile
index 876c3ef..eb19afa 100644
--- a/Makefile
+++ b/Makefile
@@ -3,7 +3,11 @@ BOARD ?= STM32F469DISC
FLAVOR ?= SPECTER
USER_C_MODULES ?= ../../../usermods
MPY_DIR ?= f469-disco/micropython
-MPY_CFLAGS ?= -Wno-dangling-pointer -Wno-enum-int-mismatch
+ifeq ($(shell uname),Linux)
+ MPY_CFLAGS ?= -Wno-dangling-pointer -Wno-enum-int-mismatch
+else
+ MPY_CFLAGS ?=
+endif
FROZEN_MANIFEST_DISCO ?= ../../../../manifests/disco.py
FROZEN_MANIFEST_DEBUG ?= ../../../../manifests/debug.py
FROZEN_MANIFEST_UNIX ?= ../../../../manifests/unix.py
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.