Skip to content

Commit 03aa02f

Browse files
committed
Rework some build_system/utils return value
Signed-off-by: dvermd <[email protected]>
1 parent aa51e18 commit 03aa02f

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

build_system/src/utils.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,7 @@ pub fn run_command_with_output(
112112
cwd: Option<&Path>,
113113
) -> Result<(), String> {
114114
let exit_status = exec_command(input, cwd, None)?;
115-
check_exit_status(input, cwd, exit_status, None, true)?;
116-
Ok(())
115+
check_exit_status(input, cwd, exit_status, None, true)
117116
}
118117

119118
pub fn run_command_with_output_and_env(
@@ -122,8 +121,7 @@ pub fn run_command_with_output_and_env(
122121
env: Option<&HashMap<String, String>>,
123122
) -> Result<(), String> {
124123
let exit_status = exec_command(input, cwd, env)?;
125-
check_exit_status(input, cwd, exit_status, None, true)?;
126-
Ok(())
124+
check_exit_status(input, cwd, exit_status, None, true)
127125
}
128126

129127
#[cfg(not(unix))]
@@ -133,8 +131,7 @@ pub fn run_command_with_output_and_env_no_err(
133131
env: Option<&HashMap<String, String>>,
134132
) -> Result<(), String> {
135133
let exit_status = exec_command(input, cwd, env)?;
136-
check_exit_status(input, cwd, exit_status, None, false)?;
137-
Ok(())
134+
check_exit_status(input, cwd, exit_status, None, false)
138135
}
139136

140137
pub fn cargo_install(to_install: &str) -> Result<(), String> {

0 commit comments

Comments
 (0)