Elyes Haouas has submitted this change. ( https://review.coreboot.org/c/coreboot/+/70251 )
(
10 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: Makefile.inc: Use 'Wmissing-include-dirs' command option ......................................................................
Makefile.inc: Use 'Wmissing-include-dirs' command option
This is to warn if a user add to Makefile a path to nonexistent directory.
Change-Id: I5a30c3830f30509deaaadc6eaeab0e17bc08565c Signed-off-by: Elyes Haouas ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/70251 Reviewed-by: Erik van den Bogaert ebogaert@eltan.com Reviewed-by: Arthur Heymans arthur@aheymans.xyz Reviewed-by: Yu-Ping Wu yupingso@google.com Reviewed-by: Fred Reitberger reitbergerfred@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M Makefile.inc 1 file changed, 20 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved Yu-Ping Wu: Looks good to me, approved Erik van den Bogaert: Looks good to me, but someone else must approve Fred Reitberger: Looks good to me, but someone else must approve
diff --git a/Makefile.inc b/Makefile.inc index f1f4646..96121d4 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -423,7 +423,7 @@ CFLAGS_common += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes CFLAGS_common += -Wwrite-strings -Wredundant-decls -Wno-trigraphs -Wimplicit-fallthrough CFLAGS_common += -Wshadow -Wdate-time -Wtype-limits -Wvla -Wold-style-definition -CFLAGS_common += -Wdangling-else +CFLAGS_common += -Wdangling-else -Wmissing-include-dirs CFLAGS_common += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer CFLAGS_common += -fstrict-aliasing -ffunction-sections -fdata-sections -fno-pie ifeq ($(CONFIG_COMPILER_GCC),y)