Skip to content

Accept xz prebuilt archives - #3

Open
macayu17 wants to merge 2 commits into
pyenv:masterfrom
macayu17:fix/prebuilt-xz-archives
Open

macayu17 wants to merge 2 commits into
pyenv:masterfrom
macayu17:fix/prebuilt-xz-archives

Conversation

@macayu17

@macayu17 macayu17 commented Sep 15, 2026

Copy link
Copy Markdown
Member

Prerequisite

Description

  • Here are some details about my PR
    • Allows pythons/update.sh to accept .tar.xz prebuilt archives while retaining .tar.gz support.
    • Updates the existing updater test to use the .tar.xz format produced by pyenv-binary.
    • Fixes the Update downloads failure in the binary publishing workflow.

Tests

  • My PR adds the following unit tests (if any)
    • The updater test passes in a Linux container.
    • Both .tar.xz indexing and existing .tar.gz compatibility were checked.
    • ShellCheck and git diff --check pass.

Summary by cubic

Accepts .tar.xz prebuilt archives in pythons/update.sh while keeping .tar.gz support, fixing the binary publishing workflow's Update downloads failure. Updates the updater test to cover both archive formats.

Written for commit 4ee3a87. Summary will update on new commits.

Review in cubic

Copilot AI lite review requested due to automatic review settings September 15, 2026 03:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread pythons/test/update.sh
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