-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Feature request: Disable quick-install for specific crate in Cargo.toml #1721
Comments
I think this would be great! Thank you for considering it. Clearly, QuickInstall is wonderful to have and make available to users, especially for projects that aren't aware of One possibility I was thinking of is that, if the install of official binaries fails, |
That can be added after Cargo.toml configuration support is done |
I'm not sure when/if I would get to it, but would you accept a PR for this or do you prefer to do these kinds of features yourself (because it's easier or for any other reason)? |
Yes, I welcome any PR and contribution from cummunity! |
I have another idea, created a PR directly because the diff is tiny. Just close it if that's no good. We can just change the order of preference such that official binaries are always preferred over quickinstall ones. |
Just to say this explicitly: while #1741 is very helpful, and decreases the urgency as far as If a project distributes official binaries via |
Yeah I agree, probably time to focus on cargo-binstall in my free time |
At this point, I don't think this is urgent, but it would be nice. Thank
you for maintaining `binstall`, it is so very useful.
|
* Update doc in SUPPORT.md for disabled-strategies For #1828 and #1721 Signed-off-by: Jiahao XU <30436523+NobodyXu@users.noreply.github.com> * Update SUPPORT.md Signed-off-by: Jiahao XU <30436523+NobodyXu@users.noreply.github.com> --------- Signed-off-by: Jiahao XU <30436523+NobodyXu@users.noreply.github.com>
Another issue is that we only provide
musl
binaries, so perhapsbinstall
prefers QuickInstall because it hasgnu
binaries.Originally posted by @ilyagr in #19 (comment)
The text was updated successfully, but these errors were encountered: