Skip to content

tools/ci: upgrade esptool on CI Dockerfile#18696

Merged
xiaoxiang781216 merged 1 commit intoapache:masterfrom
fdcavalcanti:feature/upgrade-esptool
Apr 9, 2026
Merged

tools/ci: upgrade esptool on CI Dockerfile#18696
xiaoxiang781216 merged 1 commit intoapache:masterfrom
fdcavalcanti:feature/upgrade-esptool

Conversation

@fdcavalcanti
Copy link
Copy Markdown
Contributor

Summary

  • tools/ci: upgrade esptool on CI Dockerfile

Upgrades esptool from v4.8.dev4 to v5.2.0, enabling support for newer chips.
Also adds 'upgrade pip' before installing any pip packages, which is required for this esptool upgrade.

This solves CI issue on CMake builds.

Impact

Impact on user: No.

Impact on build: New esptool version adds support for new Espressif chips.

Impact on hardware: No.

Impact on documentation: No.

Impact on security: No.

Impact on compatibility: No.

Testing

Dockerfile built locally.

Upgrades esptool from v4.8.dev4 to v5.2.0, enabling support for newer chips.
Also adds 'upgrade pip' to allow this upgrade.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
@cederom
Copy link
Copy Markdown
Contributor

cederom commented Apr 9, 2026

Hmm esptool version 5 has different cli syntax, will this impact all make flash and similar scripts or these need update too @fdcavalcanti ? :-)

Is it safe to merge @lupyuen @simbit18 ? :-)

@github-actions github-actions bot added Area: CI Size: XS The size of the change in this PR is very small labels Apr 9, 2026
@fdcavalcanti
Copy link
Copy Markdown
Contributor Author

fdcavalcanti commented Apr 9, 2026

Hmm esptool version 5 has different cli syntax, will this impact all make and similar scripts or these need update too @fdcavalcanti ? :-)

Is it safe to merge @lupyuen @simbit18 ? :-)

Should not impact. Current syntax still works but shows a warning that they will be deprecated in future releases.

@xiaoxiang781216 xiaoxiang781216 merged commit 46d2b59 into apache:master Apr 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: CI Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants