-
Notifications
You must be signed in to change notification settings - Fork 38
Description
In response to #230, you kindly added ARM64 binaries for aarch64-apple-darwin, aarch64-pc-windows-msvc, and aarch64-unknown-linux-gnu. However, aarch64-unknown-linux-musl was not included in that expansion.
I'm trying to enable cargo-auditable from within cargo-dist in my project, but I have customized the build setup a bit to build musl targets in a dedicated Rust Alpine container. That causes a failure in the dist-generated workflow that tries to install cargo-auditable with the install script. The installer script's select_archive_for_arch() has no case for aarch64-unknown-linux-musl-dynamic (or -static), so it hard-errors on that platform:
there isn't a download for your platform aarch64-unknown-linux-musl-dynamic
The x86_64-unknown-linux-musl target works fine since a prebuilt binary exists for that triple, so I assume the musl target for aarch64 was just not used by anyone up until now.
The fix should be a one-line addition to the targets list in Cargo.toml, the same change PR #231 made for the other ARM64 triples. I'm happy to submit a PR if that's helpful.