Skip to content
Snippets Groups Projects
Unverified Commit 2d1780b1 authored by GyDi's avatar GyDi Committed by GitHub
Browse files

fix: adjsut open cmd error

parent d9ce9988
No related branches found
No related tags found
No related merge requests found
......@@ -348,24 +348,26 @@ fn open_path_cmd(path: PathBuf, err_str: &str) -> Result<(), String> {
match result {
Ok(child) => match child.wait_with_output() {
Ok(out) => {
// 退出码不为0 不一定没有调用成功
// 因此仅做warn log且不返回错误
if let Some(code) = out.status.code() {
if code != 0 {
log::error!(
"failed to open path {:?} for {} (code {code})",
&path,
log::warn!("failed to open {:?} (code {})", &path, code);
log::warn!(
"open cmd stdout: {}, stderr: {}",
String::from_utf8_lossy(&out.stdout),
String::from_utf8_lossy(&out.stderr),
);
return Err(err_str.into());
}
}
}
Err(err) => {
log::error!("failed to open path {:?} for {err}", &path);
log::error!("failed to open {:?} for {err}", &path);
return Err(err_str.into());
}
},
Err(err) => {
log::error!("failed to open path {:?} for {err}", &path);
log::error!("failed to open {:?} for {err}", &path);
return Err(err_str.into());
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment