| Message ID | 1421080646-26230-1-git-send-email-ivo.slanina@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
Ivo, All, On 2015-01-12 17:37 +0100, Ivo Slanina spake thusly: > Patches located at https:// scheme URL were threated as directories, > causing build failures. > > Fixed by adding https:// pattern. > > Signed-off-by: Ivo Slanina <ivo.slanina@gmail.com> Yep, sounds legit. ;-) Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Regards, Yann E. MORIN. > --- > linux/linux.mk | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/linux/linux.mk b/linux/linux.mk > index 680521c..d02b02b 100644 > --- a/linux/linux.mk > +++ b/linux/linux.mk > @@ -147,7 +147,7 @@ endif # BR2_LINUX_KERNEL_VMLINUX > define LINUX_DOWNLOAD_PATCHES > $(if $(LINUX_PATCHES), > @$(call MESSAGE,"Download additional patches")) > - $(foreach patch,$(filter ftp://% http://%,$(LINUX_PATCHES)),\ > + $(foreach patch,$(filter ftp://% http://% https://%,$(LINUX_PATCHES)),\ > $(call DOWNLOAD,$(patch))$(sep)) > endef > > @@ -155,7 +155,7 @@ LINUX_POST_DOWNLOAD_HOOKS += LINUX_DOWNLOAD_PATCHES > > define LINUX_APPLY_PATCHES > for p in $(LINUX_PATCHES) ; do \ > - if echo $$p | grep -q -E "^ftp://|^http://" ; then \ > + if echo $$p | grep -q -E "^ftp://|^http://|^https://" ; then \ > $(APPLY_PATCHES) $(@D) $(DL_DIR) `basename $$p` ; \ > elif test -d $$p ; then \ > $(APPLY_PATCHES) $(@D) $$p linux-\*.patch ; \ > -- > 1.7.10.4 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Dear Ivo Slanina, On Mon, 12 Jan 2015 17:37:25 +0100, Ivo Slanina wrote: > Patches located at https:// scheme URL were threated as directories, > causing build failures. > > Fixed by adding https:// pattern. > > Signed-off-by: Ivo Slanina <ivo.slanina@gmail.com> > --- > linux/linux.mk | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Both patches applied, thanks. Thomas
diff --git a/linux/linux.mk b/linux/linux.mk index 680521c..d02b02b 100644 --- a/linux/linux.mk +++ b/linux/linux.mk @@ -147,7 +147,7 @@ endif # BR2_LINUX_KERNEL_VMLINUX define LINUX_DOWNLOAD_PATCHES $(if $(LINUX_PATCHES), @$(call MESSAGE,"Download additional patches")) - $(foreach patch,$(filter ftp://% http://%,$(LINUX_PATCHES)),\ + $(foreach patch,$(filter ftp://% http://% https://%,$(LINUX_PATCHES)),\ $(call DOWNLOAD,$(patch))$(sep)) endef @@ -155,7 +155,7 @@ LINUX_POST_DOWNLOAD_HOOKS += LINUX_DOWNLOAD_PATCHES define LINUX_APPLY_PATCHES for p in $(LINUX_PATCHES) ; do \ - if echo $$p | grep -q -E "^ftp://|^http://" ; then \ + if echo $$p | grep -q -E "^ftp://|^http://|^https://" ; then \ $(APPLY_PATCHES) $(@D) $(DL_DIR) `basename $$p` ; \ elif test -d $$p ; then \ $(APPLY_PATCHES) $(@D) $$p linux-\*.patch ; \
Patches located at https:// scheme URL were threated as directories, causing build failures. Fixed by adding https:// pattern. Signed-off-by: Ivo Slanina <ivo.slanina@gmail.com> --- linux/linux.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)