Add cargo-binstall support and adopt Rust target triple naming #1390
+123
−18
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds support for
cargo-binstalland transitions release assets to use Rust's standard target triple naming convention.Changes
Updated
.github/workflows/cd.yml:Updated
Cargo.toml:package.metadata.binstallconfiguration pointing to thefullvariantUpdated installation documentation:
cargo-binstallas the recommended installation methodMigration Plan
Testing
Users will be able to install spotifyd with:
cargo binstall spotifydTesting Note
Full end-to-end testing of
cargo-binstallfunctionality requires a release with the new asset naming format. The workflow changes have been validated for syntax and logic, but actual installation testing will be possible after the first release with these changes.I'm happy to help verify the installation works once a test release is created.
Closes #1200