Re: [PATCH bpf-next 3/4] tools/resolve_btfids: Set srctree variable unconditionally

2021-02-05 Thread Andrii Nakryiko
On Fri, Feb 5, 2021 at 4:46 AM Jiri Olsa wrote: > > We want this clean to be called from tree's root Makefile, > which defines same srctree variable and that will screw > the make setup. > > We actually do not use srctree being passed from outside, > so we can solve this by setting current srctree

[PATCH bpf-next 3/4] tools/resolve_btfids: Set srctree variable unconditionally

2021-02-05 Thread Jiri Olsa
We want this clean to be called from tree's root Makefile, which defines same srctree variable and that will screw the make setup. We actually do not use srctree being passed from outside, so we can solve this by setting current srctree value directly. Also changing the way how srctree is initial

Re: [PATCH bpf-next 3/4] tools/resolve_btfids: Set srctree variable unconditionally

2021-02-04 Thread Andrii Nakryiko
On Thu, Feb 4, 2021 at 1:21 PM Jiri Olsa wrote: > > We want this clean to be called from tree's root Makefile, > which defines same srctree variable and that will screw > the make setup. > > We actually do not use srctree being passed from outside, > so we can solve this by setting current srctree

Re: [PATCH bpf-next 3/4] tools/resolve_btfids: Set srctree variable unconditionally

2021-02-04 Thread Song Liu
> On Feb 4, 2021, at 1:18 PM, Jiri Olsa wrote: > > We want this clean to be called from tree's root Makefile, > which defines same srctree variable and that will screw > the make setup. > > We actually do not use srctree being passed from outside, > so we can solve this by setting current src

[PATCH bpf-next 3/4] tools/resolve_btfids: Set srctree variable unconditionally

2021-02-04 Thread Jiri Olsa
We want this clean to be called from tree's root Makefile, which defines same srctree variable and that will screw the make setup. We actually do not use srctree being passed from outside, so we can solve this by setting current srctree value directly. Also root Makefile does not define the impli