Skip to content

Conversation

@jctoledo
Copy link
Owner

@jctoledo jctoledo commented Feb 8, 2026

Summary

  • Adds --partition-table ../../partitions.csv to the cargo runner config so cargo espflash flash --release --monitor works without manual flags
  • Updates sdkconfig.defaults comment to reflect that espflash handles the partition table via flag, not ESP-IDF config

Context

The firmware binary is ~1.5MB, which exceeds the default ESP32 partition layout (1MB app). The CI workflow already passes --partition-table to espflash save-image (18a0cd5), but the local
cargo runner config was never updated to match. This caused cargo espflash flash to fail with image_too_big on every local flash attempt.

Test plan

  • cargo espflash flash --release --monitor flashes successfully without manual --partition-table flag

The CI workflow already passes --partition-table to espflash save-image
(18a0cd5), but the local cargo runner was still using the default
partition layout, causing flash failures for the 1.4MB firmware binary.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jctoledo jctoledo merged commit 658e58e into main Feb 8, 2026
5 checks passed
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