Skip to content

Handle .part extension when extracting downloads#561

Merged
SergioGasquez merged 1 commit intoesp-rs:mainfrom
SergioGasquez:fix/buil-error
Apr 17, 2026
Merged

Handle .part extension when extracting downloads#561
SergioGasquez merged 1 commit intoesp-rs:mainfrom
SergioGasquez:fix/buil-error

Conversation

@SergioGasquez
Copy link
Copy Markdown
Member

@SergioGasquez SergioGasquez commented Apr 17, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the download extraction flow by correctly handling resumable downloads saved with a trailing .part extension, so archives like *.zip.part and *.tar.xz.part can be extracted successfully.

Changes:

  • Update archive-extension detection to gracefully handle .part files by inspecting the underlying stem extension.
  • Avoid panics when the archive path has no extension or has a non-UTF8 extension (now returns a controlled “unsupported extension” error).
  • Document the fix in the changelog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/toolchain/mod.rs Detects real archive type when the downloaded file ends with .part, enabling extraction of resumed downloads.
CHANGELOG.md Adds a “Fixed” entry describing the resumed-download extraction fix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SergioGasquez SergioGasquez merged commit a2d751a into esp-rs:main Apr 17, 2026
21 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.

espup install fails with unsupported_file_extension: 'part' during Xtensa Rust installation

2 participants