[coreboot] New patch to review for coreboot: e62854c superiotool: Don't compile with -Werror

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Tue Sep 13 10:50:48 CEST 2011

Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/210


commit e62854c3fad7f47dee91a2119f46bece5a7f6cb4
Author: Mathias Krause <mathias.krause at secunet.com>
Date:   Wed Mar 9 11:37:16 2011 +0100

    superiotool: Don't compile with -Werror
    Older libpci version have headers using 'long long' which isn't allowed
    in ANSI C. Since we cannot control the libpci version installed in the
    system nor in generall have complete control over system headers, simply
    skip using -Werror in our makefile.
    Change-Id: Ibc1e57bef033bf4971f4108d078222dcf168d5e3
    Signed-off-by: Mathias Krause <mathias.krause at secunet.com>
 util/superiotool/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/util/superiotool/Makefile b/util/superiotool/Makefile
index 38a4127..6f3cfa3 100644
--- a/util/superiotool/Makefile
+++ b/util/superiotool/Makefile
@@ -29,7 +29,7 @@ PREFIX  = /usr/local
 SVNDEF := -D'SUPERIOTOOL_VERSION="$(shell svnversion -cn . \
           | sed -e "s/.*://" -e "s/\([0-9]*\).*/\1/")"'
-CFLAGS += -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing \
+CFLAGS += -O2 -Wall -Wstrict-prototypes -Wundef -Wstrict-aliasing \
          -Werror-implicit-function-declaration -ansi -pedantic $(SVNDEF)
 LDFLAGS += -lz

More information about the coreboot mailing list