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

Upgrade toolchains #2787

Merged
merged 6 commits into from
Jun 3, 2024
Merged

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jun 2, 2024

This PR updates the embedded toolchains to more recent versions, setting the new default installed version for Esp32 to IDF 5.2. The library testing github actions script has been updated to include missing esp32c2 architecture and test against IDF 4.4 and 5.2.

Esp32 IDF SDK

The installation scripts now install IDF version 5.2 by default (see issue #2784 regarding docker build failure because of Y2038 warning with IDF < 5).
Deprecation warnings printed for IDF 4.3 and 4.4 after final link step. See #2732 for discussion regarding old versions.
Windows CI tests for IDF 4.3, 5.0 removed.

Extend library tests to include missing esp32 variants

CI testing when developing libraries is much more efficient than running the main Sming CI tests.
Looks like these haven't been updated for a while so have added the esp32s3 and esp32c2 variants
and testing against IDF 5.2 for all esp32 variants. Previously was only 4.4 (old default).

Esp8266 Toolchain

The installer has been updated to use the latest toolchain (Feb 23), gcc 10.3.

RP2040 Toolchain

The installer has been updated to use the latest toolchain (Oct 23), gcc 13.2.

mikee47 added 4 commits June 1, 2024 21:09
Added `libc_replacements` which stubs-out various unused newlib functions.
Without this, linker generates warnings like `_close is not implemented and will always fail`.
These functions aren't actually used anyway and are discarded during garbage collection.

Windows install unchanged; best solution is not to use Windows.
@mikee47 mikee47 force-pushed the feature/toolchains-upgrade branch from 04d89a7 to 385c393 Compare June 2, 2024 14:28
@SmingHub SmingHub deleted a comment from what-the-diff bot Jun 2, 2024
@mikee47 mikee47 force-pushed the feature/toolchains-upgrade branch from 3209fe3 to 724168a Compare June 3, 2024 06:54
@slaff slaff added this to the 5.2.0 milestone Jun 3, 2024
@slaff slaff merged commit 4753ef4 into SmingHub:develop Jun 3, 2024
38 checks passed
@mikee47 mikee47 deleted the feature/toolchains-upgrade branch June 3, 2024 09:37
@slaff slaff mentioned this pull request Jul 4, 2024
5 tasks
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.

2 participants