[LinuxBIOS] Patch file for Fedora 8 compile problems

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Tue Dec 18 21:51:12 CET 2007


On 16.12.2007 02:39, ron minnich wrote:
> OK, I am looking at this patch again, and it's still not quite the
> right idea. Setting this kind of build option in the Config file is
> not really right.
>
> However ...
>
> if you just retained this much:
> Index: src/arch/i386/Config.lb
> ===================================================================
> --- src/arch/i386/Config.lb     (revision 3003)
> +++ src/arch/i386/Config.lb     (working copy)
> @@ -100,7 +100,7 @@
>
>          makerule linuxbios
>                 depends "crt0.o init.o $(LINUXBIOS_APC)
> $(LINUXBIOS_RAM_ROM) ldscript.ld"
> -               action  "$(CC) -nostdlib -nostartfiles -static -o $@ -T ldscript.ld crt0.o init.o"
> +               action  "$(CC) $(BUILDID_OPTION) -nostdlib -nostartfiles -static -o $@ -T ldscript.ld crt0.o init.o"
>                 action  "$(CROSS_COMPILE)nm -n linuxbios | sort > linuxbios.map"
>          end
>
>
> Then couldn't the build pick up an environment variable?
>
> Still thinking about how to fix this. I don't think what I've seen so
> far will work.
>   

Why not simply compile a small test.c file, look whether the output file
contains buildid, and set a Make variable based on that?

Regards,
Carl-Daniel




More information about the coreboot mailing list