[coreboot-gerrit] New patch to review for coreboot: Makefile: Explicitly set C99 as language standard

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Tue Nov 29 10:41:51 CET 2016


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17636

-gerrit

commit f15ac90e17e897874955c6f6fd70f01be8fc0496
Author: Paul Menzel <pmenzel at molgen.mpg.de>
Date:   Tue Nov 29 10:30:44 2016 +0100

    Makefile: Explicitly set C99 as language standard
    
    Different compiler versions use a different C language standard.
    
    GCC 4.9 uses C89, while GCC 5.x uses C99 [1].
    
    So explicitly set it to C99 as discussed on the mailing list in thread
    *[RFC] Setting C99 by default*.
    
    [1] https://www.coreboot.org/pipermail/coreboot/2016-November/082541.html
    
    Change-Id: If1569618f8044925ff72dcf3543480b34d4f90d6
    Signed-off-by: Paul Menzel <pmenzel at molgen.mpg.de>
---
 Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.inc b/Makefile.inc
index a193158..189de40 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -353,7 +353,7 @@ CPPFLAGS_common += -I$(VBOOT_SOURCE)/firmware/include
 CPPFLAGS_common += -include $(src)/include/kconfig.h
 CPPFLAGS_common += -I3rdparty
 
-CFLAGS_common += -pipe -g -nostdinc
+CFLAGS_common += -pipe -g -nostdinc -std=c99
 CFLAGS_common += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes
 CFLAGS_common += -Wwrite-strings -Wredundant-decls -Wno-trigraphs
 CFLAGS_common += -Wstrict-aliasing -Wshadow -Wdate-time



More information about the coreboot-gerrit mailing list