From 424a575cde64fe4d594930d5ef62e6f79b97b5c4 Mon Sep 17 00:00:00 2001 From: "Yann E. MORIN" Date: Wed, 13 Sep 2023 00:15:50 +0200 Subject: [PATCH] support/download/git: properly catch failures Since commit b7efb43e86da (download/git: try to recover from utterly-broken repositories), we catch errors through an ERR trap, so we can try and recover from a broken repository. In that commit, we switched from using "set -e" to "set -E", so that trap is inherited in functions, command substitutions, and subshells. However, the trap is not defined until we have parsed the options, created the cache directory, and eventually chdir()ed into it. Athough improbable, it is possible for the git helper to fail in any of those steps, and that would not get caught. Fix that Signed-off-by: Yann E. MORIN Signed-off-by: Thomas Petazzoni (cherry picked from commit daa341cb9bc82d8bcb175c46da1ba80fd9203a98) Signed-off-by: Peter Korsgaard --- support/download/git | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/support/download/git b/support/download/git index 1a1c315f73..729c553f8f 100755 --- a/support/download/git +++ b/support/download/git @@ -5,8 +5,8 @@ # (e.g. tar options, compression ratio or method)), we MUST update the format # version in the variable BR_FMT_VERSION_git, in package/pkg-download.mk. -# We want to catch any unexpected failure, and exit immediately -set -E +# We want to catch any unexpected failure +set -e # Download helper for git, to be called from the download wrapper script # @@ -77,6 +77,7 @@ pushd "${git_cache}" >/dev/null # Any error now should try to recover trap _on_error ERR +set -E # Caller needs to single-quote its arguments to prevent them from # being expanded a second time (in case there are spaces in them)