[LEDE-DEV] [PATCH] include/u-boot.mk: rename to HOST_CPPFLAGS

Felix Fietkau nbd at nbd.name
Wed Jun 14 05:23:01 PDT 2017


On 2017-06-14 02:16, Matthias Fritzsche wrote:
> * On other places the variable is written with an underscore.
> * With one $ sign it treats it as variable. With the former
>   four signs it tried to execute a command with this name.
> 
> Signed-off-by: Matthias Fritzsche <txt.file at txtfile.eu>
> ---
>  include/u-boot.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/u-boot.mk b/include/u-boot.mk
> index 4b3ff69d6b..6cfc05c76c 100644
> --- a/include/u-boot.mk
> +++ b/include/u-boot.mk
> @@ -42,7 +42,7 @@ TARGET_DEP = TARGET_$(BUILD_TARGET)$(if $(BUILD_SUBTARGET),_$(BUILD_SUBTARGET))
>  
>  UBOOT_MAKE_FLAGS = \
>  	HOSTCC="$(HOSTCC)" \
> -	HOSTCFLAGS='$(HOST_CFLAGS) $$$$(HOSTCPPFLAGS)' \
> +	HOSTCFLAGS='$(HOST_CFLAGS) $(HOST_CPPFLAGS)' \
Did you actually test this? I believe it will break several u-boot
builds. The intention behind $$$$(HOSTCPPFLAGS) was to let the submake
inside the build system resolve this to its own HOSTCPPFLAGS variable.

- Felix



More information about the Lede-dev mailing list