[PATCH] buildrom/filo build fix

[Daniel Mack]

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; \

[Myles Watson]

> buildrom does not currently build filo due to a missing include path.
> --- Makefile	(revision 94)

I just bumped the revision in buildrom to 105.  I'd been using 104
successfully, and 105 still builds, so I bumped it to 105.  Hopefully you
can build and use filo without changing it now.

Thanks,
Myles

[Stefan Reinauer]

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?

Stefan

[Daniel Mack]

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