[LEDE-DEV] [PATCH 2/2] include/kernel: Switch to git download method

[Felix Fietkau]

On 2016-11-09 18:34, Florian Fainelli wrote:
> Utilize the existing git download logic from include/download.mk and migrate
> the kernel download over to it. This avoids repeatedly cloning kernel sources
> after a make target/linux/clean for instance.
> 
> Signed-off-by: Florian Fainelli <f.fainelli at gmail.com>
> ---
>  include/kernel-build.mk    | 19 +++++++++++++++++++
>  include/kernel-defaults.mk | 11 +----------
>  2 files changed, 20 insertions(+), 10 deletions(-)
> 
> diff --git a/include/kernel-build.mk b/include/kernel-build.mk
> index 04878f8250a9..04c6fc1e1986 100644
> --- a/include/kernel-build.mk
> +++ b/include/kernel-build.mk
> @@ -46,6 +46,24 @@ define Download/kernel
>    MD5SUM:=$(LINUX_KERNEL_MD5SUM)
>  endef
>  
> +KERNEL_GIT_OPTS:=
> +ifneq ($(strip $(CONFIG_KERNEL_GIT_LOCAL_REPOSITORY)),"")
> +  KERNEL_GIT_OPTS+=--reference $(CONFIG_KERNEL_GIT_LOCAL_REPOSITORY)
> +endif
> +
> +ifneq ($(strip $(CONFIG_KERNEL_GIT_BRANCH)),"")
> +  KERNEL_GIT_OPTS+=--branch $(CONFIG_KERNEL_GIT_BRANCH)
> +endif
> +
> +define Download/git-kernel
> +  URL:=$(CONFIG_KERNEL_GIT_CLONE_URI)
> +  PROTO:=git
> +  VERSION:=$(CONFIG_KERNEL_GIT_BRANCH)
> +  FILE:=$(LINUX_SOURCE)
> +  SUBDIR:=linux-$(KERNEL_PATCHVER)
> +  OPTS:=$(KERNEL_GIT_OPTS)
> +endef
> +
>  ifdef CONFIG_COLLECT_KERNEL_DEBUG
>    define Kernel/CollectDebug
>  	rm -rf $(KERNEL_BUILD_DIR)/debug
> @@ -73,6 +91,7 @@ endif
>  define BuildKernel
>    $(if $(QUILT),$(Build/Quilt))
>    $(if $(LINUX_SITE),$(call Download,kernel))
> +  $(if $(CONFIG_KERNEL_GIT_CLONE_URI),$(call Download,git-kernel))
This caused a build error when not using external git trees because you
forgot to strip quotes. I've fixed it up in my staging tree, please
check the result.

- Felix