b70ce56651
Commit 54d3d94b6e ("support/download: print
command used for download") broke the git and svn download helpers, because
these helpers have invocations of the _git/_svn commands where the exact
output matters.
For example for git, this would result in:
date: invalid date ‘GIT_DIR=.../dl/libyuv/git/.git git log -1 --pretty=format:%ci \n2019-04-12 17:48:45 +0000’
Detected a corrupted git cache.
Removing it and starting afresh.
Fix by splitting the _git function in two: _git and _plain_git.
The former echoes the command, and then calls the latter.
Most invocations use _git as before, but those cases where the output should
not be disturbed, directly call _plain_git.
For symmetry, all download helpers are aligned, even though only the git and
svn helpers were broken.
Fixes: #13631
Fixes:
http://autobuild.buildroot.org/results/c2f/c2fcd4aa6660e3c2f9c6f85646ca7dfe0db56040/
Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
[yann.morin.1998@free.fr: add bug report and autobuild failure]
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
51 lines
1.4 KiB
Bash
Executable File
51 lines
1.4 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
|
|
# We want to catch any unexpected failure, and exit immediately
|
|
set -e
|
|
|
|
# Download helper for wget, to be called from the download wrapper script
|
|
#
|
|
# Options:
|
|
# -q Be quiet.
|
|
# -o FILE Save into file FILE.
|
|
# -f FILENAME The filename of the tarball to get at URL
|
|
# -u URL Download file at URL.
|
|
# -e ENCODE Tell wget to urlencode the filename passed to it
|
|
#
|
|
# Environment:
|
|
# WGET : the wget command to call
|
|
|
|
quiet=
|
|
while getopts "${BR_BACKEND_DL_GETOPTS}" OPT; do
|
|
case "${OPT}" in
|
|
q) quiet=-q;;
|
|
o) output="${OPTARG}";;
|
|
f) filename="${OPTARG}";;
|
|
u) url="${OPTARG}";;
|
|
e) encode="-e";;
|
|
:) printf "option '%s' expects a mandatory argument\n" "${OPTARG}"; exit 1;;
|
|
\?) printf "unknown option '%s'\n" "${OPTARG}" >&2; exit 1;;
|
|
esac
|
|
done
|
|
|
|
shift $((OPTIND-1)) # Get rid of our options
|
|
|
|
# Caller needs to single-quote its arguments to prevent them from
|
|
# being expanded a second time (in case there are spaces in them)
|
|
_wget() {
|
|
if [ -z "${quiet}" ]; then
|
|
printf '%s ' ${WGET} "${@}"; printf '\n'
|
|
fi
|
|
_plain_wget "$@"
|
|
}
|
|
# Note: please keep command below aligned with what is printed above
|
|
_plain_wget() {
|
|
eval ${WGET} "${@}"
|
|
}
|
|
|
|
# Replace every '?' with '%3F' in the filename; only for the PRIMARY and BACKUP
|
|
# mirror
|
|
[ -n "${encode}" ] && filename=${filename//\?/%3F}
|
|
|
|
_wget ${quiet} "${@}" -O "'${output}'" "'${url}/${filename}'"
|