From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?S=C3=A9bastien=20Szymanski?= Date: Wed, 25 May 2016 14:13:59 +0200 Subject: [Buildroot] [PATCH 1/1] support/scripts/apply-patches.sh: handle tar and mkdir failures Message-ID: <1464178439-3333-1-git-send-email-sebastien.szymanski@armadeus.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net With several archives in a space-separated list of patches, if one archive cannot be untar (i.e because it is not readable), Buildroot keeps going instead of aborting. Fix this by checking the tar return code. While at it, check if the unpack archive directory creation is successful. Signed-off-by: S?bastien Szymanski --- support/scripts/apply-patches.sh | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/support/scripts/apply-patches.sh b/support/scripts/apply-patches.sh index 201278d..b0a8ad4 100755 --- a/support/scripts/apply-patches.sh +++ b/support/scripts/apply-patches.sh @@ -132,7 +132,11 @@ function scan_patchdir { unpackedarchivedir="$builddir/.patches-$(basename $i)-unpacked" rm -rf "$unpackedarchivedir" 2> /dev/null mkdir "$unpackedarchivedir" - tar -C "$unpackedarchivedir" -xaf "${path}/$i" + if [ $? != 0 ] ; then + echo "Error: cannot create $unpackedarchivedir directory." + exit 1 + fi + tar -C "$unpackedarchivedir" -xaf "${path}/$i" || exit 1 scan_patchdir "$unpackedarchivedir" else apply_patch "$path" "$i" -- 2.3.6