Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add cargo-run-bin to readme #1514

Closed
passcod opened this issue Dec 8, 2023 · 6 comments · Fixed by #1524
Closed

Add cargo-run-bin to readme #1514

passcod opened this issue Dec 8, 2023 · 6 comments · Fixed by #1524

Comments

@passcod
Copy link
Member

passcod commented Dec 8, 2023

https://dustinblackman.com/posts/why-does-everyone-install-crates-globally/

This essentially implements the manifest idea and also leaves binstall to focus on, well, binstalling. Win win! Let's add it to the readme and maybe other docs so we can finally put this idea to bed.

@NobodyXu
Copy link
Member

NobodyXu commented Dec 8, 2023

Wow that's a really nice piece of software from @dustinblackman

I will create a PR to put it in README

@passcod
Copy link
Member Author

passcod commented Dec 8, 2023

@dustinblackman is there anything binstall could do to make run-bin's life easier?

@dustinblackman
Copy link
Contributor

dustinblackman commented Dec 8, 2023

@dustinblackman is there anything binstall could do to make run-bin's life easier?

Nope! It does it's job and it does it well. Because you've put effort in to matching both Cargo's inputs and log responses, I have to do very little to make everything work nicely with binstall.

@NobodyXu
Copy link
Member

NobodyXu commented Dec 8, 2023

        .arg("--install-path")
        .arg(cache_path.join("bin"));

I don't think this is necessary since you already specifies the root.

@NobodyXu
Copy link
Member

NobodyXu commented Dec 8, 2023

Another recommendation is to run cargo-binstall to install multiple binaries at once, cargo-binstall would be able to reuse http connections and install them in parallel, though this is not as important as correctness.

@dustinblackman
Copy link
Contributor

Appreciate the feedback! I'll take a look :)

passcod added a commit that referenced this issue Dec 13, 2023
- Reorders sections to show Binstall's value upfront
- Clarifies and simplifies language
- Updates the sample output
- Includes cargo-run-bin (#1514)
- Places the github actions in more prominence in the Install section
- Unifies appellation by consistently using the "Binstall" name except for commands
- Changes the arch labels in the install table to reflect common/official usage
@passcod passcod mentioned this issue Dec 13, 2023
github-merge-queue bot pushed a commit that referenced this issue Dec 13, 2023
- Reorders sections to show Binstall's value upfront
- Clarifies and simplifies language
- Updates the sample output
- Includes cargo-run-bin (#1514)
- Places the github actions in more prominence in the Install section
- Unifies appellation by consistently using the "Binstall" name except for commands
- Changes the arch labels in the install table to reflect common/official usage
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 a pull request may close this issue.

3 participants