Skip to content

build: add cargo-binstall support#288

Merged
MordechaiHadad merged 1 commit into
MordechaiHadad:masterfrom
PeterCardenas:support-cargo-binstall
Jun 26, 2025
Merged

build: add cargo-binstall support#288
MordechaiHadad merged 1 commit into
MordechaiHadad:masterfrom
PeterCardenas:support-cargo-binstall

Conversation

@PeterCardenas

@PeterCardenas PeterCardenas commented Jun 26, 2025

Copy link
Copy Markdown
Contributor

Support cargo binstall by adding the manifest fields to the Cargo.toml according to this documentation. cargo binstall reads the manifest to automatically install binaries instead of building. Currently cargo-binstall will install from quick install which installs incorrect binaries on macos (x86_64 instead of arm).

An alternative to changing the manifest is to change the names of the release artifacts so that it can be automatically detected by cargo binstall. Let me know if this is preferred.

Tested by running and verifying architecture with file:

  • cargo binstall --force --manifest-path . --root . bob-nvim, installed the arm version on my mac correctly
  • cargo binstall --force --targets <target> --manifest-path . --root . bob-nvim
    • Following values for <target> were tested:
      • aarch64-apple-darwin
      • x86_64-apple-darwin
      • x86_64-pc-windows-msvc
      • x86_64-unknown-linux-gnu

@MordechaiHadad

Copy link
Copy Markdown
Owner

Really cool!

@MordechaiHadad MordechaiHadad merged commit 452be03 into MordechaiHadad:master Jun 26, 2025
27 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.

2 participants