[coreboot] [PATCH] buildrom/filo build fix

Daniel Mack daniel at caiaq.de
Wed Sep 23 11:54:32 CEST 2009


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




More information about the coreboot mailing list