Skip to content

Update framework-portduino version in platform.json#16

Merged
thebentern merged 1 commit into
developfrom
bump-framework-portduino-wifi-fix
Apr 26, 2026
Merged

Update framework-portduino version in platform.json#16
thebentern merged 1 commit into
developfrom
bump-framework-portduino-wifi-fix

Conversation

@thebentern

@thebentern thebentern commented Apr 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Bumps the framework-portduino git ref from 45b678b to 4473291 (the merge of meshtastic/framework-portduino#68) so this platform picks up the WiFi submodule's extern "C" / logging.h fix in WiFiServer.cpp.

What's in the bump

4473291 — "Bump libraries/WiFi to pick up extern "C" / logging.h fix" (framework-portduino#68). Single submodule pointer change: c75a5e8467ec17 in libraries/WiFi. That submodule commit (meshtastic/WiFi#5) moves #include "logging.h" out of WiFiServer.cpp's extern "C" { ... } block so arduino::log(...) keeps C++ linkage and doesn't collide with libc's extern "C" double log(double) from <math.h> under Apple Clang.

Context

Hit while bringing up the macOS-native portduino build — see firmware PR meshtastic/firmware#10300.

Downstream chain

Test plan

  • No-op on Linux: pure pin bump; the framework-side change is preprocessor-order only.
  • Verified equivalent transformation builds meshtasticd cleanly on macOS arm64 — the firmware-side pre-build patcher applies the same reordering and the resulting binary boots in SimRadio mode.

🤖 Generated with Claude Code

@thebentern thebentern marked this pull request as ready for review April 26, 2026 02:10
@thebentern thebentern force-pushed the bump-framework-portduino-wifi-fix branch from 0ff8a93 to 132a59a Compare April 26, 2026 02:10
@thebentern thebentern merged commit 2a8fd4f into develop Apr 26, 2026
6 checks passed
@thebentern thebentern deleted the bump-framework-portduino-wifi-fix branch April 26, 2026 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant