diff mbox

[OpenWrt-Devel] build: download.mk: bugfix download.pl argument call order

Message ID 1453362586-16199-1-git-send-email-Eckert.Florian@googlemail.com
State Changes Requested
Headers show

Commit Message

Florian Eckert Jan. 21, 2016, 7:49 a.m. UTC 
From: Florian Eckert <Eckert.florian@googlemail.come>

Signed-off-by: Florian Eckert <Eckert.Florian@googlemail.com>
---
 include/download.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

John Crispin Jan. 21, 2016, 7:52 a.m. UTC | #1 
On 21/01/2016 08:49, Florian Eckert wrote:
> From: Florian Eckert <Eckert.florian@googlemail.come>
> 

NAK, the description is missing




> Signed-off-by: Florian Eckert <Eckert.Florian@googlemail.com>
> ---
>  include/download.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/download.mk b/include/download.mk
> index ca5eee4..562a7b9 100644
> --- a/include/download.mk
> +++ b/include/download.mk
> @@ -48,7 +48,7 @@ define DownloadMethod/default
>  endef
>  
>  define wrap_mirror
> -	$(if $(if $(MIRROR),$(filter-out x,$(MIRROR_MD5SUM))),@$(SCRIPT_DIR)/download.pl "$(DL_DIR)" "$(FILE)" "" "$(MIRROR_MD5SUM)" || ( $(1) ),$(1))
> +	$(if $(if $(MIRROR),$(filter-out x,$(MIRROR_MD5SUM))),@$(SCRIPT_DIR)/download.pl "$(DL_DIR)" "$(FILE)" "$(MIRROR_MD5SUM)" "" || ( $(1) ),$(1))
>  endef
>  
>  define DownloadMethod/cvs
>
diff mbox

Patch

diff --git a/include/download.mk b/include/download.mk
index ca5eee4..562a7b9 100644
--- a/include/download.mk
+++ b/include/download.mk
@@ -48,7 +48,7 @@  define DownloadMethod/default
 endef
 
 define wrap_mirror
-	$(if $(if $(MIRROR),$(filter-out x,$(MIRROR_MD5SUM))),@$(SCRIPT_DIR)/download.pl "$(DL_DIR)" "$(FILE)" "" "$(MIRROR_MD5SUM)" || ( $(1) ),$(1))
+	$(if $(if $(MIRROR),$(filter-out x,$(MIRROR_MD5SUM))),@$(SCRIPT_DIR)/download.pl "$(DL_DIR)" "$(FILE)" "$(MIRROR_MD5SUM)" "" || ( $(1) ),$(1))
 endef
 
 define DownloadMethod/cvs