[coreboot] buildrom bug

[Myles Watson]

On Thu, Apr 24, 2008 at 8:38 AM, Jordan Crouse <jordan.crouse at amd.com> wrote:
> On 24/04/08 09:54 -0400, Ward Vandewege wrote:
>  > On Thu, Apr 24, 2008 at 12:54:20PM +0200, Peter Stuge wrote:
>  > > On Thu, Apr 24, 2008 at 11:21:41AM +0200, Uwe Hermann wrote:
>  > > > On Thu, Apr 24, 2008 at 06:32:17AM +0200, Peter Stuge wrote:
>  > > > > I configured v2 and TINT, then:
>  > > >
>  > > > Cannot reproduce using ALIX.1C target in buildrom.
>  > >
>  > > Maybe you have ruby installed.
>  > >
>  > >
>  > > > Which target did you use?
>  > >
>  > > I thought I picked 1C but it seems I was using DB800.
>  > >
>  > >
>  > > > Do you have some .config* .tmpconfig* .kconfig* files around?
>  > > > Try a fresh checkout, just in case.
>  > >
>  > > It was a fresh checkout. Am retrying.
>  > >
>  > >
>  > > > > $ make
>  > > > > /home/stuge/co/buildrom/buildrom-devel/packages/grub2/grub2.mk:28:
>  > > > > *** To build GRUB2, you need to install 'ruby'.  Stop.
>  > >
>  > > Still the same.
>  >
>  >
>  > It's a bug. This is a workaround (I'm still working on a proper fix,
>  > make should not include all payload makefiles all the time):

This patch changes the Makefile and the config/payloads/*.conf files so that
only the payload .mk files that are needed are included.  It still includes
kernel.mk, roms.mk and geodevsa.mk every time, but that seemed like a separate
change.

Thanks,
Myles

Signed-off-by: Myles Watson <mylesgw at gmail.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: payload_handling.patch
Type: text/x-patch
Size: 3188 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080425/31fe58ef/attachment.patch>