From ce2cd2553a937d84f1a78386ab28502e89f247d6 Mon Sep 17 00:00:00 2001 From: tv Date: Fri, 13 Nov 2026 20:36:10 +0000 Subject: populate: exit with the status of the command that failed The shell implementation runs under set -e, so it exits with the status of the command that failed, e.g. 255 if ssh can't reach the target, which wrappers retrying on connection failures rely on, or 128 if git fetch fails. The program does the same, instead of exiting with 1, and with 128 plus the signal for a command a signal ended. A phase's expect.status is the exit status it expects populate to fail with; git-status checks git fetch's. --- pkgs/populate/src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'pkgs/populate/src/main.rs') diff --git a/pkgs/populate/src/main.rs b/pkgs/populate/src/main.rs index 17d9678..2884fd0 100644 --- a/pkgs/populate/src/main.rs +++ b/pkgs/populate/src/main.rs @@ -78,7 +78,7 @@ fn exit(failure: run::Failure) -> ExitCode { eprintln!("{message}"); ExitCode::FAILURE } - run::Failure::Command => ExitCode::FAILURE, + run::Failure::Command(status) => ExitCode::from(status), run::Failure::Signal(signal) => ExitCode::from(128 + signal as u8), } } -- cgit v1.3.1