[LinuxBIOS] Patch file for ld/fedora 8 issue as promised...

Myles Watson myles at pel.cs.byu.edu
Mon Jan 7 17:37:44 CET 2008


On Jan 7, 2008 8:15 AM, Marc Karasek <Marc.Karasek at sun.com> wrote:
> After looking at the script Myles sent, I immediately saw the problem.
> I forgot that if  [ $build_id ] will always be true because it checks to
> see if it is defined not the value of build_id.  My bad, sorry.
>
> I have made the changes and added an == 1 to the if statement.  Attached
> is the new and I hope final patch file for this.
>
It works for me (it doesn't add the load option.)

Sorry to be picky, but it seems like this breaks if they mention
build-id more than once in the help in the future.  I think >0 would
be better than ==1.

With that fixed, or if no one thinks that will ever happen:
Acked-by: Myles Watson <myles at pel.cs.byu.edu>
>
> /*********************
> Marc Karasek
> MTS
> Sun Microsystems
> mailto:marc.karasek at sun.com
> ph:770.360.6415
> *********************/
>
>
>
> Ed Swierk wrote:
>
> > On 1/4/08, Marc Karasek <Marc.Karasek at sun.com> wrote:
> >
> >> I made a test script and ran it and it sets build_id = 1 properly.  I
> >> have also included this script.
> >>
> >
> > The problem is that on Planet Bourne, zero means true and nonzero means false.
> >
> > --Ed
> >
>




More information about the coreboot mailing list