From: Yann E. MORIN Date: Sun, 31 Jul 2016 22:35:55 +0000 (+0200) Subject: support/download: make the git wrapper more robust X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4f87aa7db5ee;p=buildroot.git support/download: make the git wrapper more robust Currently, there are two failure paths in the wrapper: - if the tar fails, then the error is ignored because it is on the left-hand-side of a pipe; - if the find fails, then the error is ignored because it is a process substitution (and there is a pipe, too). While the former could be fixed with "set -o pipefail", the latter can not be fixed thusly and we must use an intemediate file for it. So, fix both issues by using intermediate files, both to generate the list of files to include in the archive, and generate the archive in a temporary tarball. Fixes the following build issue, where the find is failing for whatever unknown reason: http://autobuild.buildroot.net/results/20f/20fd76d2256eee81837f7e9bbaefbe79d7645ae9/ And this one, where the process substitution failed, also for an unknown reason: http://autobuild.buildroot.org/results/018/018971ea9227b386fe25d3c264c7e80b843a9f68/ Signed-off-by: "Yann E. MORIN" Signed-off-by: Thomas Petazzoni --- diff --git a/support/download/git b/support/download/git index 281db61a9a..7921411835 100755 --- a/support/download/git +++ b/support/download/git @@ -92,6 +92,8 @@ rm -rf .git popd >/dev/null # Generate the archive, sort with the C locale so that it is reproducible +find "${basename}" -not -type d >"${basename}.list" +LC_ALL=C sort <"${basename}.list" >"${basename}.list.sorted" tar cf - --numeric-owner --owner=0 --group=0 --mtime="${date}" \ - -T <(find "${basename}" -not -type d |LC_ALL=C sort) \ -|gzip -n >"${output}" + -T "${basename}.list.sorted" >"${output}.tar" +gzip -n <"${output}.tar" >"${output}"