rkvm icon indicating copy to clipboard operation
rkvm copied to clipboard

add `[[bin]] name` to Cargo.toml to output prefixed executables

Open colemickens opened this issue 4 years ago • 2 comments
trafficstars

In the course of packaging this, I noticed that the built binaries are called client, server, and certificate-gen.

For my Nix derivation, I simply prefix them all with rkvm- but I'm wondering if it would be more appropriate to just fix this in the various binaries' Cargo.toml files so that they're automatically named with a prefix.

What do you think?

colemickens avatar Jan 11 '21 23:01 colemickens

I forgot, the other reason this would be nice, I could add the same GitHub action that I added for another Rust project and you'd get Windows binaries attached to a GHA job for each commit. That way you can get more Windows users without them having to build it.

(versus, a zip of "client.exe, server.exe, certificate-gen.exe" might be a bit weird)

colemickens avatar Jan 12 '21 00:01 colemickens

This is also the case for cargo install --path server. It ends up putting a binary called server on your path. So I also think it would be great to rename it. I'm not sure if @htrefil might be more responsive if you just open a PR and maybe they'll just merge it

BrendanBall avatar Jan 16 '21 06:01 BrendanBall

Fixed.

htrefil avatar Apr 18 '23 20:04 htrefil