[Buildroot] [PATCH] dl-wrapper: Fix urlencode option never being sent to backend

[Damien Thébault]

The urlencode parameter is lost in dl-wrapper, because we try to get it
from backend instead of backend_urlencode.

We must get the urlencode information from backend_urlencode.

Signed-off-by: Damien Thébault <***@vitec.com>
---
support/download/dl-wrapper | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/download/dl-wrapper b/support/download/dl-wrapper
index bb70c98253..3315bd410e 100755
--- a/support/download/dl-wrapper
+++ b/support/download/dl-wrapper
@@ -93,7 +93,7 @@ main() {
esac
uri=${uri#*+}

- urlencode=${backend#*|}
+ urlencode=${backend_urlencode#*|}
# urlencode must be "urlencode"
[ "${urlencode}" != "urlencode" ] && urlencode=""

--
2.19.1

[Yann E. MORIN]

Damien, All,

Post by Damien Thébault
The urlencode parameter is lost in dl-wrapper, because we try to get it
from backend instead of backend_urlencode.
We must get the urlencode information from backend_urlencode.

What is this trying to fix, actually? We have no download issue in the
autobuilders, as far as I can see... Do you have a test-case to share?

But true, this is incorrect, and was broken by commit 38de434123
(download: fix file:// BR2_PRIMARY_SITE (download cache)).

Reviewed-by: "Yann E. MORIN" <***@free.fr>

Regards,
Yann E. MORIN.

Post by Damien Thébault
---
support/download/dl-wrapper | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/support/download/dl-wrapper b/support/download/dl-wrapper
index bb70c98253..3315bd410e 100755
--- a/support/download/dl-wrapper
+++ b/support/download/dl-wrapper
@@ -93,7 +93,7 @@ main() {
esac
uri=${uri#*+}
- urlencode=${backend#*|}
+ urlencode=${backend_urlencode#*|}
# urlencode must be "urlencode"
[ "${urlencode}" != "urlencode" ] && urlencode=""
--
2.19.1
_______________________________________________
buildroot mailing list
http://lists.busybox.net/mailman/listinfo/buildroot

[Damien Thébault]

Post by Yann E. MORIN

Post by Damien Thébault
The urlencode parameter is lost in dl-wrapper, because we try to get it
from backend instead of backend_urlencode.
We must get the urlencode information from backend_urlencode.

What is this trying to fix, actually? We have no download issue in
the autobuilders, as far as I can see... Do you have a test-case to
share?

Hello Yann,

I have a build configured with a primary site and
BR2_PRIMARY_SITE_ONLY=y that failed to download libopenssl patches from
the mirror.

If the autobuilders are directly using the original URLs or are falling
back to the original URLs they should work fine.

Post by Yann E. MORIN
Applied to master, thanks.

Thank you !
--
Damien Thébault

[Yann E. MORIN]

Damien, All,

Post by Damien Thébault

Post by Yann E. MORIN

Post by Damien Thébault
The urlencode parameter is lost in dl-wrapper, because we try to get it
from backend instead of backend_urlencode.
We must get the urlencode information from backend_urlencode.

What is this trying to fix, actually? We have no download issue in
the autobuilders, as far as I can see... Do you have a test-case to
share?

I have a build configured with a primary site and
BR2_PRIMARY_SITE_ONLY=y that failed to download libopenssl patches from
the mirror.

OK, makes sense, thanks for the feedback. :-)

Regards,
Yann E. MORIN.

[Thomas Petazzoni]

Hello,

Post by Damien Thébault
I have a build configured with a primary site and
BR2_PRIMARY_SITE_ONLY=y that failed to download libopenssl patches from
the mirror.
If the autobuilders are directly using the original URLs or are falling
back to the original URLs they should work fine.

Thanks! We really need to add some test cases in support/testing/ for
those "corner" cases, since they are not tested by the autobuilders or
in the most usual situations, they tend to be forgotten, and therefore
broken.

Thomas

[Thomas Petazzoni]

Hello,

Post by Damien Thébault
The urlencode parameter is lost in dl-wrapper, because we try to get it
from backend instead of backend_urlencode.
We must get the urlencode information from backend_urlencode.
---
support/download/dl-wrapper | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas