Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Esp32 build fixes #2836

Merged
merged 5 commits into from
Jun 25, 2024
Merged

Esp32 build fixes #2836

merged 5 commits into from
Jun 25, 2024

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jun 24, 2024

Further to #2819 the size mainly seems to relate to two things:

  1. WPA3 support is enabled by default, but probably isn't necessary for most applications (could be wrong).
  2. More debug stuff in IDF 5.2; release builds are comparable to IDF 4.4.

Bug fixes

  • esp32 timer definitions missing in RELEASE build
  • sdkconfig definitions not picked up

Size fixes

Disable WPA3 by default

This is also enabled by default in IDF 4.4 but can always be re-enabled if required.
It adds quite a lot to the image size.

Don't pull in stdio code from newlib

Sming doesn't use newlib stdio streams

Others

  • Revise CI tool scanlog.py to include missing builds (HostTests)
  • Make checksdk a pre-requisite

@slaff slaff added this to the 5.2.0 milestone Jun 25, 2024
@slaff slaff merged commit f69ea9c into SmingHub:develop Jun 25, 2024
48 checks passed
@mikee47 mikee47 deleted the fix/esp32-build-size branch June 25, 2024 12:23
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.

None yet

2 participants