non archived binaries are not executable

[fgardt]

I wanted to get clippy-sarif with cargo-binstall and used the described way to get unsupported crates like so:

cargo binstall \ 
    --pkg-url="{ repo }/releases/download/{ name }-v{ version }/{ name }-{ target }" \
    --pkg-fmt="bin" clippy-sarif

note: for crates from sarif-rs it would've been nice to allow to get multiple crates since they all use the same overrides for --pkg-url and --pkg-fmt

The binary installs just fine at the right location etc but when trying to run it you can't because it is not marked executable.
It would be nice if cargo-binstall would mark binaries as executable so I don't need to run chmod +x -R ~/.cargo/bin

[NobodyXu]

note: for crates from sarif-rs it would've been nice to allow to get multiple crates since they all use the same overrides for --pkg-url and --pkg-fmt

I think that is possible.
Just provides multiple crate name.

[NobodyXu]

It would be nice if cargo-binstall would mark binaries as executable so I don't need to run chmod +x -R ~/.cargo/bin

Sure!

[fgardt]

note: for crates from sarif-rs it would've been nice to allow to get multiple crates since they all use the same overrides for --pkg-url and --pkg-fmt

I think that is possible.
Just provides multiple crate name.

Are you saying that this is already possible or that it should be easy to add?
Because it is not currently possible as far as I can tell:

[NobodyXu]

Are you saying that this is already possible or that it should be easy to add?

It's definitely possible.
The error were introduced when we added support for multiple binaries, we can adjust that for this use case.

[NobodyXu]

@fgardt cargo-binstall v0.14.0 has released which fixed this issue.

note: for crates from sarif-rs it would've been nice to allow to get multiple crates since they all use the same overrides for --pkg-url and --pkg-fmt

I forgot to implement this in this release, would do that in next release.
Anyway, I would create a new issue for it in #444.