Messages in this thread | | | Date | Wed, 18 Sep 2019 14:09:08 +0300 | From | Ivan Khoronzhuk <> | Subject | Re: [PATCH v3 bpf-next 13/14] samples: bpf: makefile: add sysroot support |
| |
On Tue, Sep 17, 2019 at 10:23:57PM -0700, Andrii Nakryiko wrote: >On Mon, Sep 16, 2019 at 4:00 AM Ivan Khoronzhuk ><ivan.khoronzhuk@linaro.org> wrote: >> >> Basically it only enables that was added by previous couple fixes. >> Sysroot contains correct libs installed and its headers ofc. Useful > >Please, let's not use unnecessary abbreviations/slang. "Of course" is >not too long and is a proper English, let's stick to it. > >> when working with NFC or virtual machine. >> >> Usage: >> >> clean (on demand) >> make ARCH=arm -C samples/bpf clean >> make ARCH=arm -C tools clean >> make ARCH=arm clean >> >> configure and install headers: >> >> make ARCH=arm defconfig >> make ARCH=arm headers_install >> >> build samples/bpf: >> make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- samples/bpf/ \ >> SYSROOT="path/to/sysroot" >> >> Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> >> --- >> samples/bpf/Makefile | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/samples/bpf/Makefile b/samples/bpf/Makefile >> index 133123d4c7d7..57ddf055d6c3 100644 >> --- a/samples/bpf/Makefile >> +++ b/samples/bpf/Makefile >> @@ -194,6 +194,11 @@ TPROGS_CFLAGS += -I$(srctree)/tools/lib/ >> TPROGS_CFLAGS += -I$(srctree)/tools/include >> TPROGS_CFLAGS += -I$(srctree)/tools/perf >> >> +ifdef SYSROOT >> +TPROGS_CFLAGS += --sysroot=${SYSROOT} >> +TPROGS_LDFLAGS := -L${SYSROOT}/usr/lib > >Please stay consistent: $() instead of ${}? Yes, thanks.
> >> +endif >> + >> EXTRA_CXXFLAGS := $(TPROGS_CFLAGS) >> >> # options not valid for C++ >> -- >> 2.17.1 >>
-- Regards, Ivan Khoronzhuk
| |