Re: [PATCH] Add --sysroot when building dlfilters on cross build environments

From: Adrian Hunter
Date: Tue Aug 31 2021 - 02:05:38 EST


On 31/08/21 12:12 am, Arnaldo Carvalho de Melo wrote:
> Hi Adrian,
>
> I had to add this cset to fix the build/link on cross build
> environments with a sysroot, like Android, EXTRA_CFLAGS is where it will
> get the needed --sysroot=/opt/android-ndk-r12b//platforms/android-24/arch-arm, please ack.
>
> - Arnaldo
>
> diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
> index 24623599113df215..e04313c4d8409a94 100644
> --- a/tools/perf/Makefile.perf
> +++ b/tools/perf/Makefile.perf
> @@ -785,10 +785,10 @@ endif
>
> $(OUTPUT)dlfilters/%.o: dlfilters/%.c include/perf/perf_dlfilter.h
> $(Q)$(MKDIR) -p $(OUTPUT)dlfilters
> - $(QUIET_CC)$(CC) -c -Iinclude -o $@ -fpic $<
> + $(QUIET_CC)$(CC) -c -Iinclude $(EXTRA_CFLAGS) -o $@ -fpic $<
>
> $(OUTPUT)dlfilters/%.so: $(OUTPUT)dlfilters/%.o
> - $(QUIET_LINK)$(CC) -shared -o $@ $<
> + $(QUIET_LINK)$(CC) $(EXTRA_CFLAGS) -shared -o $@ $<
>
> ifndef NO_JVMTI
> LIBJVMTI_IN := $(OUTPUT)jvmti/jvmti-in.o
>

Acked-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>