Skip to content

Commit f8d092c

Browse files
zulinx86roypat
authored andcommitted
fix(cpu-template-helper): Don't show error message twice
The CPU template helper tool showed its error message twice as follows: ``` $cpu-template-helper fingerprint compare --prev A.json --curr B.json Difference detected between source and target:\\n{ "name": "kernel_version", "prev": "5.10.215-203.850.amzn2.x86_64", "curr": "6.1.84-99.169.amzn2023.x86_64" } Error: FingerprintCompare(DiffDetected("{\n \"name\": \"kernel_...")) ``` It is because it returned `Result` from `main()` to return an exit code and it shows its error message on error. Instead just return the exit code using `std::process::exit()`. Signed-off-by: Takahiro Itazuri <[email protected]>
1 parent afad05d commit f8d092c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/cpu-template-helper/src/main.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,14 +190,14 @@ fn run(cli: Cli) -> Result<(), HelperError> {
190190
Ok(())
191191
}
192192

193-
fn main() -> Result<(), HelperError> {
193+
fn main() {
194194
let cli = Cli::parse();
195195
let result = run(cli);
196196
if let Err(e) = result {
197197
eprintln!("{}", e);
198-
Err(e)
198+
std::process::exit(1);
199199
} else {
200-
Ok(())
200+
std::process::exit(0);
201201
}
202202
}
203203

0 commit comments

Comments
 (0)