Skip to content

Exclude web server, paxcounter and few others from original ESP32 generation to fix IRAM overflow#10005

Merged
thebentern merged 6 commits into
masterfrom
exclude_web
Mar 25, 2026
Merged

Exclude web server, paxcounter and few others from original ESP32 generation to fix IRAM overflow#10005
thebentern merged 6 commits into
masterfrom
exclude_web

Conversation

@thebentern

Copy link
Copy Markdown
Contributor

Sad, but original ESP32 can't support webserver anymore due to IRAM constraints

@thebentern thebentern requested review from Copilot and vidplace7 March 25, 2026 14:18
@github-actions github-actions Bot added needs-review Needs human review bugfix Pull request that fixes bugs labels Mar 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates ESP32 (original/“OG”) build configuration to reduce IRAM usage by excluding higher-footprint features (notably the HTTP web server and paxcounter), and adds a compile-time stub for the web server header when the web server is excluded.

Changes:

  • Exclude Paxcounter, web server (mesh/http), and related features from the OG ESP32 build via build_src_filter and MESHTASTIC_EXCLUDE_* flags.
  • Adjust OG ESP32 lib_deps to remove web server / paxcounter-related dependencies.
  • Add stub declarations/definitions in WebServer.h for builds where the web server is excluded.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
variants/esp32/station-g1/platformio.ini Unflags BOARD_HAS_PSRAM for Station G1 environment.
variants/esp32/esp32.ini Excludes web server/paxcounter sources and sets MESHTASTIC_EXCLUDE_* flags for OG ESP32; updates dependencies.
src/mesh/http/WebServer.h Adds MESHTASTIC_EXCLUDE_WEBSERVER guard and stubbed APIs/types when excluded.

Comment thread variants/esp32/esp32.ini Outdated
Comment thread variants/esp32/esp32.ini Outdated
thebentern and others added 5 commits March 25, 2026 11:00
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Removed WiFiAPClient.cpp from build source filter.
@thebentern thebentern merged commit c4bac04 into master Mar 25, 2026
101 of 133 checks passed
@thebentern thebentern deleted the exclude_web branch March 25, 2026 22:52
skrashevich pushed a commit to skrashevich/meshtastic-firmware that referenced this pull request Apr 4, 2026
…eration to fix IRAM overflow (meshtastic#10005)

* Exclude web server, paxcounter and few others from original ESP32 generation to fix IRAM overflow

* Update variants/esp32/esp32.ini

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update variants/esp32/esp32.ini

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update build source filter in esp32.ini

Removed WiFiAPClient.cpp from build source filter.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
cpatulea referenced this pull request in dbinfrago/libpax May 5, 2026
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
…eration to fix IRAM overflow (meshtastic#10005)

* Exclude web server, paxcounter and few others from original ESP32 generation to fix IRAM overflow

* Update variants/esp32/esp32.ini

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update variants/esp32/esp32.ini

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update build source filter in esp32.ini

Removed WiFiAPClient.cpp from build source filter.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes bugs needs-review Needs human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants