summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYann E. MORIN <yann.morin.1998@free.fr>2018-04-04 18:22:39 +0200
committerPeter Korsgaard <peter@korsgaard.com>2018-04-06 12:43:47 +0200
commit585c5dc35a0354452b7836c5449fa43715a5d0cc (patch)
tree338556ce6ba2a70168b9d0afa4202fb8e2a2cba1
parentc261f44b953535c0eb0cbd907f0b079205e81d88 (diff)
downloadbuildroot-585c5dc35a0354452b7836c5449fa43715a5d0cc.tar.gz
buildroot-585c5dc35a0354452b7836c5449fa43715a5d0cc.zip
core/download fix local backend
Since c8ef0c03b0b (download: put most of the infra in dl-wrapper), the backend for local files is now named after the scheme, which is 'file' for a local file. >From the same commit on, the directory part and the basename are now passed separately, to let the backend reconstruct the full path when it needs to do so, which is the case for the 'file' backend too. Finaly, ff559846fdc1 (support/download: Add support to pass options directly to downloaders) introduced a nasty error, as it made use of "${@}" when calling its internal function. Revert that mess now... Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com> Cc: Peter Korsgaard <peter@korsgaard.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Cc: Arnout Vandecappelle <arnout@mind.be> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
-rwxr-xr-xsupport/download/file (renamed from support/download/cp)9
1 files changed, 6 insertions, 3 deletions
diff --git a/support/download/cp b/support/download/file
index 52fe2de83d..fefd6d271c 100755
--- a/support/download/cp
+++ b/support/download/file
@@ -2,13 +2,15 @@
# We want to catch any unexpected failure, and exit immediately
set -e
+set -x
# Download helper for cp, to be called from the download wrapper script
#
# Options:
# -q Be quiet.
# -o FILE Copy to file FILE.
-# -u FILE Copy from file FILE.
+# -f FILE Copy from basename file FILE.
+# -u DIR Copy from FILE in DIR.
#
# Environment:
# LOCALFILES: the cp command to call
@@ -23,7 +25,8 @@ while getopts "${BR_BACKEND_DL_GETOPTS}" OPT; do
case "${OPT}" in
q) verbose=;;
o) output="${OPTARG}";;
- u) source="${OPTARG}";;
+ f) file="${OPTARG}";;
+ u) dir="${OPTARG}";;
:) printf "option '%s' expects a mandatory argument\n" "${OPTARG}"; exit 1;;
\?) printf "unknown option '%s'\n" "${OPTARG}" >&2; exit 1;;
esac
@@ -37,4 +40,4 @@ _localfiles() {
eval ${LOCALFILES} "${@}"
}
-_localfiles ${verbose} "${@}""'${source}'" "'${output}'"
+_localfiles ${verbose} "'${dir}${file}'" "'${output}'"
OpenPOWER on IntegriCloud