Hello Felix Singer, build bot (Jenkins), Nico Huber, Patrick Georgi, Martin Roth, Paul Menzel, Angel Pons, HAOUAS Elyes,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/47153
to look at the new patch set (#4).
Change subject: libpayload: Add compiler.h to compiler parameters ......................................................................
libpayload: Add compiler.h to compiler parameters
Headers in libpayload define various structs like so:
struct struct_name { ... } __packed;
However, these header files do not include the compiler.h macro that defines what __packed is, so they are actually defining a variable named __packed and *not* declaring a packed struct. This leads to defining the same variable multiple times, which was caught by GCC 10. Add compiler.h to the compiler parameters so it is included in all files automatically.
Signed-off-by: Jacob Garber jgarber1@ualberta.ca Change-Id: Ia67182520dc94149e06fe9e03a14b3fc2ee29973 --- M payloads/libpayload/Makefile.inc M payloads/libpayload/bin/lpgcc M payloads/libpayload/include/cbfs_core.h M payloads/libpayload/include/libpayload.h 4 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/47153/4