Skip to content

Fetch the cargo-dist binary directly instead of using the installer#18731

Merged
zanieb merged 1 commit intomainfrom
zb/harden-dist-install
Mar 26, 2026
Merged

Fetch the cargo-dist binary directly instead of using the installer#18731
zanieb merged 1 commit intomainfrom
zb/harden-dist-install

Conversation

@zanieb
Copy link
Copy Markdown
Member

@zanieb zanieb commented Mar 26, 2026

No description provided.

@zanieb zanieb added the internal A refactor or improvement that is not user-facing label Mar 26, 2026
Copy link
Copy Markdown
Contributor

@Gankra Gankra left a comment

Choose a reason for hiding this comment

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

Increasingly comitting to never removing dirty but yeah this works.

run: "curl --proto '=https' --tlsv1.2 -LsSf https://github.com/axodotdev/cargo-dist/releases/download/v0.31.0/cargo-dist-installer.sh | sh"
run: |
curl --proto '=https' --tlsv1.2 -LsSf "https://github.com/axodotdev/cargo-dist/releases/download/v${CARGO_DIST_VERSION}/cargo-dist-x86_64-unknown-linux-gnu.tar.xz" -o /tmp/cargo-dist.tar.xz
echo "${CARGO_DIST_CHECKSUM} /tmp/cargo-dist.tar.xz" | sha256sum -c -
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems like a very odd way of writing this but if it's the way that works...

@zanieb zanieb marked this pull request as ready for review March 26, 2026 20:31
@zanieb zanieb enabled auto-merge (squash) March 26, 2026 20:31
@zanieb zanieb disabled auto-merge March 26, 2026 20:39
@zanieb zanieb merged commit 66c6591 into main Mar 26, 2026
69 of 70 checks passed
@zanieb zanieb deleted the zb/harden-dist-install branch March 26, 2026 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants