[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