Idwer Vollering has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38648 )
Change subject: util/cbfstool: extend includes in commonlib ......................................................................
Patch Set 1:
Patch Set 1:
I'm open for suggestions. So far, adding 'TOOLCPPFLAGS += -include /usr/include/sys/endian.h' and/or 'TOOLCPPFLAGS += -I/usr/include/sys' while leaving lz4_wrapper.c (where it had the ifdef guard to use sys/endian.h instead) untouched seems to not be working.
Well, the first one wouldn't work because you still have an #include it can't resolve.
I would expect directly including this .h to work.
I'd expect the second one to work, though. What's the error message? (Do you have /usr/include/sys/endian.h on your system or is it somewhere else?)
$ find /usr/include -name endian.h -exec grep le32toh {} + /usr/include/sys/endian.h:#define le32toh(x) ((uint32_t)(x)) /usr/include/sys/endian.h:#define le32toh(x) bswap32((x))