[coreboot] [LinuxBIOS] [PATCH] buildrom: add extract andbusybox-config, uclibc-config targets

Myles Watson myles at pel.cs.byu.edu
Wed Feb 6 21:52:23 CET 2008

> I've implemented that - the naming convention for the customconfig files
> is
> now


> and in the case of uclibc and the kernel, I've added the architecture in
> there as well:
>   customconfig--$(PAYLOAD)--$(UCLIBC_ARCH)--$(COREBOOT_VENDOR)-
> Custom config files will only be used if the payload, (architecture,)
> vendor
> and board match.


> If a custom config file is used, buildrom will say so on stdout.
> The *-config commands are a bit smarter now: they will copy an existing
> custom config file back to the source directory so that doing a subsequent
> make something-config will have the effect of editing the custom config,
> rather than overwriting it.
> > I think that could be considered a matter of taste.  Maybe we should add
> the
> > custom-config option for the kernel as well.
> I've added the kernel.


It's missing the ARCH=$(KERNEL_BUILD_ARCH) on the configure line for kernel
and TARGET_ARCH=$(UCLIBC_BUILD_ARCH) for uclibc.  On my x86_64 box it makes
a 64-bit configuration even for 32-bit targets.  

I'd like it if the verbose messages about using the custom config were
printed after the configuration was done.  I never saw the messages because
make menuconfig is so verbose.

At first I was worried about the dependencies that you added like:


I thought it would have the same problem we had before with the stamp
directory being "modified" when the stamps were, and forcing a complete
rebuild, but it didn't seem to happen.  I guess I don't understand that well

Besides that it looks very good.

Acked-by: Myles Watson <myles at pel.cs.byu.edu>


> Thanks,
> Ward.
> --
> Ward Vandewege <ward at fsf.org>
> Free Software Foundation - Senior System Administrator

More information about the coreboot mailing list