[CRIU] [V2 6/6] tools.mk: support override CC/LD command line

[Dmitry Safonov]

On Tue, 2 Apr 2019 at 03:44, <ning.a.zhang at intel.com> wrote:
>
> From: Zhang Ning <ning.a.zhang at intel.com>
>
> due to Android NDK's clang is x86_64-linux-android28-clang --sysroot ${SYSROOT_PATH}
> and it's ld is x86_64-linux-android-ld,
> it's not able to use a single pattern to discript clang and ld.
>
> and there is a error for x86_64-linux-android-ld.
> x86_64-linux-android-ld  -L/home/ning/source/criu/protobuf-c/../target/lib -lprotobuf-c -r -z noexecstack -T ./compel/arch/x86/scripts/compel-pack.lds.S  -o criu/pie/parasite.built-in.o  criu/pie/parasite.o criu/pie/pie.lib.a ./compel/plugins/std.lib.a
> ./compel/compel-host hgen -f criu/pie/parasite.built-in.o -o criu/pie/parasite-blob.h
> Error (compel/src/lib/handle-elf-host.c:335): Unexpected undefined symbol: `'. External symbol in PIE?
> criu/pie/Makefile:49: recipe for target 'criu/pie/parasite-blob.h' failed
>
> rebuild with host ld, can pass build.
>
> so support override CC/LD from command line can pass build.
>
> Cc: Chen Hu <hu1.chen at intel.com>
> Signed-off-by: Zhang Ning <ning.a.zhang at intel.com>

LGTM,
Reviewed-by: Dmitry Safonov <0x7f454c46 at gmail.com>

> ---
>  scripts/nmk/scripts/tools.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/nmk/scripts/tools.mk b/scripts/nmk/scripts/tools.mk
> index 485fe9f2..d756358d 100644
> --- a/scripts/nmk/scripts/tools.mk
> +++ b/scripts/nmk/scripts/tools.mk
> @@ -4,9 +4,9 @@ ifndef ____nmk_defined__tools
>  # System tools shorthands
>  RM             := rm -f
>  HOSTLD         ?= ld
> -LD             := $(CROSS_COMPILE)$(HOSTLD)
> +LD             ?= $(CROSS_COMPILE)$(HOSTLD)
>  HOSTCC         ?= gcc
> -CC             := $(CROSS_COMPILE)$(HOSTCC)
> +CC             ?= $(CROSS_COMPILE)$(HOSTCC)
>  CPP            := $(CC) -E
>  AS             := $(CROSS_COMPILE)as
>  AR             := $(CROSS_COMPILE)ar
> --
> 2.20.1
>
> _______________________________________________
> CRIU mailing list
> CRIU at openvz.org
> https://lists.openvz.org/mailman/listinfo/criu

Thanks,
             Dmitry