actions/cargo-binstall
1
0
Fork 0
mirror of https://github.com/cargo-bins/cargo-binstall.git synced 2025-04-24 14:28:42 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix reinstalling crates after removed by cargo-uninstall (#628)

Browse source 
Honor `.crates.toml` only since `cargo-uninstall` can only update
`.crates.toml`, not binstall's own manifest.

Fixed #625

Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
This commit is contained in:
Jiahao XU 2022-12-24 16:04:51 +11:00 • committed by GitHub
parent 76e89f7b30
commit cdff92aa3c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

17
crates/bin/src/entry.rs
View file

@ -287,18 +287,11 @@ fn filter_out_installed_crates(
let name = &crate_name.name;
let curr_version = metadata
.and_then(|metadata| {
metadata
.0
.get(name)
.map(|crate_info| &crate_info.current_version)
.into_iter()
.chain(metadata.1.get(name))
// Since the cargo_install_v1_metadata could be out of sync
// from cargo_binstall_metadata, it is better to obtain
// the version from both of them and takes the larger one.
.max()
});
// `cargo-uninstall` can be called to uninstall crates,
// but it only updates .crates.toml.
//
// So here we will honour .crates.toml only.
.and_then(|metadata| metadata.1.get(name));
match (
force,