On Wed, Sep 23, 2009 at 11:43:02AM +0200, Stefan Reinauer wrote: > Daniel Mack wrote: > > buildrom does not currently build filo due to a missing include path. > > This fixes it. > > > > --- > > > > Index: Makefile > > =================================================================== > > --- Makefile (revision 94) > > +++ Makefile (working copy) > > @@ -76,7 +76,7 @@ > > LIBGCC = $(shell $(CC) $(CFLAGS) -print-libgcc-file-name) > > > > OBJS := $(patsubst %,$(obj)/%,$(TARGETS-y)) > > -INCLUDES := -I$(INCPAYLOAD) -Iinclude -I$(ARCHDIR-y)/include -Ibuild > > +INCLUDES := -I$(INCPAYLOAD) -I$(INCPAYLOAD)/$(ARCHDIR-y) -Iinclude > > -I$(ARCHDIR-y)/include -Ibuild > > INCLUDES += -I$(GCCINCDIR) > > > > try-run= $(shell set -e; \ > > > > > What's your error without this? Is that a patch against FILO?
It couldn't find arch specific includes. But it was all due to an old version of filo - the fix went in upstream with r98. Buildrom uses r104 now, so it's all fine. Sorry for the noise. Daniel -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
