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

Add error logging if achieve doesn't contain bin required (#1298)

Browse source 
* Add `error` logging if achieve doesn't contain bin required

So that users know that this is an upstream issue and will report to
upstream.

Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>

* Update crates/binstalk/src/ops/resolve.rs

---------

Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
This commit is contained in:
Jiahao XU 2023-08-17 09:13:21 +10:00 • committed by GitHub
parent 146d3f8f43
commit 0f7828569a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

11
crates/binstalk/src/ops/resolve.rs
View file

@ -14,7 +14,7 @@ use maybe_owned::MaybeOwned;
use semver::{Version, VersionReq};
use tempfile::TempDir;
use tokio::task::spawn_blocking;
use tracing::{debug, info, instrument, warn};
use tracing::{debug, error, info, instrument, warn};
use crate::{
bins,
@ -267,17 +267,22 @@ async fn download_extract_and_verify(
// This binary is optional
Err(err) => {
let required_features = &bin.required_features;
let bin_name = bin.name.as_str();
if required_features.is_empty() {
error!(
"When resolving {name} bin {bin_name} is not found.\
This binary is not optional so it must be included in the archive, please contact with\
upstream to fix this issue."
);
// This bin is not optional, error
Some(Err(err))
} else {
// Optional, print a warning and continue.
let bin_name = bin.name.as_str();
let features = required_features.iter().format(",");
warn!(
"When resolving {name} bin {bin_name} is not found. \
But since it requies features {features}, this bin is ignored."
But since it requies features {features}, this bin is ignored."
);
None
}