Skip to content

Add win-arm64 platform support to tools schema#103

Merged
ivmarkov merged 1 commit intoesp-rs:masterfrom
drinkcat:winarm64
Mar 28, 2026
Merged

Add win-arm64 platform support to tools schema#103
ivmarkov merged 1 commit intoesp-rs:masterfrom
drinkcat:winarm64

Conversation

@drinkcat
Copy link
Copy Markdown
Contributor

@drinkcat drinkcat commented Mar 28, 2026

Submission Checklist 📝

  • I have updated existing examples or added new ones (if applicable).
  • I have used cargo fmt command to ensure that all changed code is formatted correctly.
  • I have used cargo clippy command to ensure that all changed code passes latest Clippy nightly lints.
  • My changes were added to the CHANGELOG.md in the proper section.

Pull Request Details 📖

Description

The ESP-IDF tools.json manifest added a win-arm64 platform variant for Windows ARM64. Add it to PlatformOverrideInfoPlatformsItem enum, VersionInfo struct, and the OS matcher in parse_tools to prevent panics when parsing manifests that include this platform.

Testing

Spotted on the idf-next CI for esp-idf-hal (https://github.com/esp-rs/esp-idf-hal/actions/runs/23682870831/job/68997591274). It's aleady a problem on release/v6.0 (not just master), so probably worth having already.

The ESP-IDF tools.json manifest added a win-arm64 platform variant for
Windows ARM64. Add it to PlatformOverrideInfoPlatformsItem enum,
VersionInfo struct, and the OS matcher in parse_tools to prevent panics
when parsing manifests that include this platform.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ivmarkov
Copy link
Copy Markdown
Collaborator

You'll have to then temporarily [patch.crates-io] the embuild into the sys, hal and svc crates.

@ivmarkov ivmarkov merged commit aa8add5 into esp-rs:master Mar 28, 2026
1 check passed
@drinkcat
Copy link
Copy Markdown
Contributor Author

You'll have to then temporarily [patch.crates-io] the embuild into the sys, hal and svc crates.

Yeah, maybe this one is not too critical and can wait for a release... CIEspIdfNext has been red for a long time.

@drinkcat drinkcat deleted the winarm64 branch March 28, 2026 16:28
@ivmarkov
Copy link
Copy Markdown
Collaborator

You'll have to then temporarily [patch.crates-io] the embuild into the sys, hal and svc crates.

Yeah, maybe this one is not too critical and can wait for a release... CIEspIdfNext has been red for a long time.

True, but no big deal to also add embuild to the patch section.

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