Check zlib feature error about bpftool HOT 3 CLOSED

Hi, thanks a lot for the report!

Both issues are bugs in the feature detection. It looks that clang does not need the flags telling to the linker what libraries to use, and this results in warnings that break detection due to the use of -Werror. Adding Wno-unused-command-line-argument seems to fix it indeed.

I don't directly experience the issue with the echo on my setup, it “works fine for me”. But echo is not known to be portable, and I'm ready to believe that the current code breaks on other environments. I observed that the $(shell echo) invocation calls /bin/sh (dash for me) which is supposed to use /usr/bin/echo, but it seems that in some cases (including the default case for me, where the current code works), some built-in echo is used instead. I think it's some built-in from make, but didn't manage to confirm it. Anyway. Adding -e is not a solution though: it breaks on my setup. I think we'll switch to using printf instead of echo, hopefully it will be more stable.

If you have a moment, could you please check whether the changes in #5 fix the build on your setup?

from bpftool.

Hi, thanks a lot for the report!

Both issues are bugs in the feature detection. It looks that clang does not need the flags telling to the linker what libraries to use, and this results in warnings that break detection due to the use of -Werror. Adding Wno-unused-command-line-argument seems to fix it indeed.

I don't directly experience the issue with the echo on my setup, it “works fine for me”. But echo is not known to be portable, and I'm ready to believe that the current code breaks on other environments. I observed that the $(shell echo) invocation calls /bin/sh (dash for me) which is supposed to use /usr/bin/echo, but it seems that in some cases (including the default case for me, where the current code works), some built-in echo is used instead. I think it's some built-in from make, but didn't manage to confirm it. Anyway. Adding -e is not a solution though: it breaks on my setup. I think we'll switch to using printf instead of echo, hopefully it will be more stable.

If you have a moment, could you please check whether the changes in #5 fix the build on your setup?

check in my env, it's ok.
thank your replay.
... libbfd: [ on ]
... disassembler-four-args: [ OFF ]
... zlib: [ on ]
... libcap: [ OFF ]
... clang-bpf-co-re: [ on ]

from bpftool.

And thank you for testing :) Merged now.

from bpftool.