Teach rustc-fake to tell the truth about subprocess exit statuses (#1366)

yaahc · web-flow · commit 3c253134664f · 2022-07-23T03:05:17.000Z
diff --git a/collector/src/rustc-fake.rs b/collector/src/rustc-fake.rs
@@ -467,8 +467,9 @@ fn process_self_profile_output(prof_out_dir: PathBuf, args: &[OsString]) {
 #[cfg(windows)]
 fn exec(cmd: &mut Command) {
     let cmd_d = format!("{:?}", cmd);
-    if let Err(e) = cmd.status() {
-        panic!("failed to execute `{}`: {}", cmd_d, e);
+    match cmd.status() {
+        Ok(status) => std::process::exit(status.code().unwrap_or(1)),
+        Err(e) => panic!("failed to spawn `{}`: {}", cmd_d, e),
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -467,8 +467,9 @@ fn process_self_profile_output(prof_out_dir: PathBuf, args: &[OsString]) {`
`467`	`467`	`#[cfg(windows)]`
`468`	`468`	`fn exec(cmd: &mut Command) {`
`469`	`469`	`let cmd_d = format!("{:?}", cmd);`
`470`		`- if let Err(e) = cmd.status() {`
`471`		- panic!("failed to execute `{}`: {}", cmd_d, e);
	`470`	`+ match cmd.status() {`
	`471`	`+ Ok(status) => std::process::exit(status.code().unwrap_or(1)),`
	`472`	+ Err(e) => panic!("failed to spawn `{}`: {}", cmd_d, e),
`472`	`473`	`}`
`473`	`474`	`}`
`474`	`475`