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

Unfortunately this update doesn't help with the problems i was seeing on
my Debian squeeze (stable) GCC :Debian 4.4.5-8.

CC wasn't set and bison and flex were not instaled.

The acpica-unix-20110922/README states that the Makefiles contain
CC = gcc but only the main acpica-unix-20110922/generate/
unix/Makefile.config contains it and it is not used by coreboot.

The attached patch adds CC = gcc and also tests for bison and flex.

Signed-off-by: Nils Jacobs <njaco...@adsltotaal.nl>

Thanks, Nils.

--- acpica-unix-20110922/compiler/Makefile
+++ acpica-unix-20110922/compiler/Makefile
@@ -19,6 +19,7 @@
 
 HOST =          _LINUX
 NOMAN =         YES
+CC =            gcc
 COMPILE =       $(CC) -c $(CFLAGS) $(CWARNINGFLAGS) -o$@ $<
 
 ACPICA_COMPONENTS =
@@ -319,3 +320,21 @@
 
 install :
 	$(INSTALLPROG)
+
+.PHONY: FORCE
+FORCE:
+
+#
+# Test for bison and flex existance
+#
+is_flex_installed := $(shell if flex --version > /dev/null 2>& 1; then echo yes; fi)
+ifneq '$(is_flex_installed)' 'yes'
+$(warning Please install the 'flex' utility on your system)
+$(error Error: 'flex' not installed)
+endif
+
+is_bison_installed := $(shell if bison --version > /dev/null 2>& 1; then echo yes; fi)
+ifneq '$(is_bison_installed)' 'yes'
+$(warning Please install the 'bison' utility on your system)
+$(error Error: 'bison' not installed)
+endif
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to