Skip to content

Commit

Permalink
[x] don't log failing sccache stops if the client failed due to a non…
Browse files Browse the repository at this point in the history
…-running server

Closes: aptos-labs#8538
  • Loading branch information
rexhoffman authored and bors-libra committed Jun 15, 2021
1 parent 983ba00 commit c4638a6
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions devtools/x/src/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,14 @@ pub fn stop_sccache_server() {
if output.status.success() {
info!("Stopped already running sccache.");
} else {
info!("Failed to stopped already running sccache.");
warn!("status: {}", output.status);
warn!("stdout: {}", String::from_utf8_lossy(&output.stdout));
warn!("stderr: {}", String::from_utf8_lossy(&output.stderr));
let std_err = String::from_utf8_lossy(&output.stderr);
//sccache will fail
if !std_err.contains("couldn't connect to server") {
warn!("Failed to stopped already running sccache.");
warn!("status: {}", output.status);
warn!("stdout: {}", String::from_utf8_lossy(&output.stdout));
warn!("stderr: {}", std_err);
}
}
}
Err(error) => {
Expand Down

0 comments on commit c4638a6

Please sign in to comment.