mbox series

[bpf-next,0/3] fixes for bpftool's Makefile

Message ID 20211009210341.6291-1-quentin@isovalent.com (mailing list archive)
Headers show
Series fixes for bpftool's Makefile | expand

Message

Quentin Monnet Oct. 9, 2021, 9:03 p.m. UTC
This set contains one fix for bpftool's Makefile, to make sure that the
headers internal to libbpf are installed properly even if we add more
headers to the relevant Makefile variable in the future (although we'd like
to avoid that if possible).

The other patches aim at cleaning up the output from the Makefile, in
particular when running the command "make" another time after bpftool is
built.

Quentin Monnet (3):
  bpftool: fix install for libbpf's internal header(s)
  bpftool: do not FORCE-build libbpf
  bpftool: turn check on zlib from a phony target into a conditional
    error

 tools/bpf/bpftool/Makefile | 29 +++++++++++++++--------------
 1 file changed, 15 insertions(+), 14 deletions(-)

Comments

John Fastabend Oct. 18, 2021, 2:05 p.m. UTC | #1
Quentin Monnet wrote:
> This set contains one fix for bpftool's Makefile, to make sure that the
> headers internal to libbpf are installed properly even if we add more
> headers to the relevant Makefile variable in the future (although we'd like
> to avoid that if possible).
> 
> The other patches aim at cleaning up the output from the Makefile, in
> particular when running the command "make" another time after bpftool is
> built.
> 
> Quentin Monnet (3):
>   bpftool: fix install for libbpf's internal header(s)
>   bpftool: do not FORCE-build libbpf
>   bpftool: turn check on zlib from a phony target into a conditional
>     error
> 
>  tools/bpf/bpftool/Makefile | 29 +++++++++++++++--------------
>  1 file changed, 15 insertions(+), 14 deletions(-)
> 
> -- 
> 2.30.2
> 

I'm not a Makefile expert, but from my side these look good. Thanks.

Acked-by: John Fastabend <john.fastabend@gmail.com>
Andrii Nakryiko Oct. 19, 2021, 11:44 p.m. UTC | #2
On Mon, Oct 18, 2021 at 7:05 AM John Fastabend <john.fastabend@gmail.com> wrote:
>
> Quentin Monnet wrote:
> > This set contains one fix for bpftool's Makefile, to make sure that the
> > headers internal to libbpf are installed properly even if we add more
> > headers to the relevant Makefile variable in the future (although we'd like
> > to avoid that if possible).
> >
> > The other patches aim at cleaning up the output from the Makefile, in
> > particular when running the command "make" another time after bpftool is
> > built.
> >
> > Quentin Monnet (3):
> >   bpftool: fix install for libbpf's internal header(s)
> >   bpftool: do not FORCE-build libbpf
> >   bpftool: turn check on zlib from a phony target into a conditional
> >     error
> >
> >  tools/bpf/bpftool/Makefile | 29 +++++++++++++++--------------
> >  1 file changed, 15 insertions(+), 14 deletions(-)
> >
> > --
> > 2.30.2
> >
>
> I'm not a Makefile expert, but from my side these look good. Thanks.
>
> Acked-by: John Fastabend <john.fastabend@gmail.com>

Applied to bpf-next, thanks.