[coreboot-gerrit] New patch to review for coreboot: 5ea220f cbmem: Makefile: Fix typo: s, CPPFLAGS, CFLAGS,

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Fri Apr 5 11:24:35 CEST 2013


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

-gerrit

commit 5ea220f7056480a3415e548e2533d36cc18823a8
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Fri Apr 5 11:12:12 2013 +0200

    cbmem: Makefile: Fix typo: s,CPPFLAGS,CFLAGS,
    
    Commit »cbmem compilation needs to use the hardened toolchain«
    (7c6b6bb5) [1] introduced `CPPFLAGS` but used `CFLAGS`, which gives
    the following build error, which was not spotted by Jenkins,
    because programs under `utils/` are not build tested.
    
        cbmem.c:36:34: fatal error: boot/coreboot_tables.h: Datei oder Verzeichnis nicht gefunden
        compilation terminated.
    
    As this is a C program, change `CPPFLAGS` to `CFLAGS`, causing the
    actual directories get passed to the compiler and the header file
    is found then.
    
    [1] http://review.coreboot.org/1791
    
    Change-Id: I254d76c7d98b6f920501407239566e9655b2ddeb
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 util/cbmem/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbmem/Makefile b/util/cbmem/Makefile
index bf74e08..e976d28 100644
--- a/util/cbmem/Makefile
+++ b/util/cbmem/Makefile
@@ -22,7 +22,7 @@ ROOT = ../../src
 CC     = $(CROSS_COMPILE)gcc
 CFLAGS ?= -O2
 CFLAGS += -Wall -Werror
-CPPFLAGS += -iquote $(ROOT)/include -iquote $(ROOT)/src/arch/x86
+CFLAGS += -iquote $(ROOT)/include -iquote $(ROOT)/src/arch/x86
 
 OBJS = $(PROGRAM).o
 



More information about the coreboot-gerrit mailing list