[PATCH v7 00/15] build improvements

Jouni Malinen j at w1.fi
Sat Oct 10 09:55:46 EDT 2020


On Fri, Sep 18, 2020 at 11:49:46AM +0200, Johannes Berg wrote:
>  - After fixing the include directive in v6, patch 13 was
>    wrong of course, fix that ...

Thanks, applied patches other than 11/15 and 13/15:

    build: Use the new build system for fuzz tests

This seems to break building of some of the fuzzing tools. For example,
tests/fuzzing/ap-mgmt:

hostap/tests/fuzzing/ap-mgmt$ make 
cc: error: ../../../src/crypto/libcrypto.a: No such file or directory
cc: error: ../../../src/tls/libtls.a: No such file or directory
make: *** [Makefile:38: ap-mgmt] Error 1

It looks like ELIBS gets some of the build rules confused. These
libcrypto.a and libtls.a show up three times on the linker command line
and the last instance of each points to this old location. Replacing
that with the new location fixes this, but I'm not completely sure how
this was supposed to work properly, so I did not try to fix this in the
build files. These were needed on the command line at least twice to get
some mixed references included into the binary, but I do not recall
where that third instance is coming from.


      build: Put archive files into build/ folder too

This depends on the 11/15, so I had to drop this as well for now.

-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list