diff options
| author | Thomas Petazzoni <thomas.petazzoni@bootlin.com> | 2018-04-11 10:10:24 +0200 |
|---|---|---|
| committer | Thomas Petazzoni <thomas.petazzoni@bootlin.com> | 2018-04-12 22:54:00 +0200 |
| commit | 26794449807648284439c6e03a405418d34f523f (patch) | |
| tree | 54f9665f6b6ebf6df38f8775302954345320acb7 | |
| parent | 0cf077a209596f34a109b641c90d85a87876f605 (diff) | |
| download | buildroot-26794449807648284439c6e03a405418d34f523f.tar.gz buildroot-26794449807648284439c6e03a405418d34f523f.zip | |
support/download/dl-wrapper: pass the correct -N option
${raw_name} is never defined in dl-wrapper, and therefore the value
passed to the -N option is always empty. This causes a problem for the
'cvs' backend, which uses the value of this option as the CVS module
to be downloaded.
If the name of the CVS module is omitted, all the CVS modules from
that CVS repository are downloaded, which creates a tarball with a lot
more contents, and the actual useful contents in a sub-directory,
obviously breaking patches that should be applied, and the entire
build process that follows.
Fixes:
http://autobuild.buildroot.net/results/fcee0e3d7eeeb373313b1794092c729b1b052348/
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
| -rwxr-xr-x | support/download/dl-wrapper | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/support/download/dl-wrapper b/support/download/dl-wrapper index 3d2118a4ef..8d6365e08d 100755 --- a/support/download/dl-wrapper +++ b/support/download/dl-wrapper @@ -125,7 +125,7 @@ main() { -c "${cset}" \ -d "${dl_dir}" \ -n "${raw_base_name}" \ - -N "${raw_name}" \ + -N "${base_name}" \ -f "${filename}" \ -u "${uri}" \ -o "${tmpf}" \ |
