[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 3/3] tools/libs: Fix headers.chk logic



On 05.03.21 13:49, Andrew Cooper wrote:
c/s 4664034cd dropped the $(LIBHEADERSGLOB) dependency for the headers.chk
rule, without replacing it.

As headers.chk uses $^, a typical build looks like:

   andrewcoop@andrewcoop:/local/xen.git$ make -C tools/libs/devicemodel/
   make: Entering directory '/local/xen.git/tools/libs/devicemodel'
   for i in ; do \
       gcc -x c -ansi -Wall -Werror
       -I/local/xen.git/tools/libs/devicemodel/../../../tools/include \
             -S -o /dev/null $i || exit 1; \
       echo $i; \
   done >headers.chk.new
   mv headers.chk.new headers.chk

i.e. with an empty for loop, and checking only the $(AUTOINCS).

Reinsert a $(LIBHEADERS) dependency.

Fixes: 4664034cd ("tools/libs: move official headers to common directory")
Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: application/pgp-keys

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.