> -----Original Message-----
> From: Corey Osgood [mailto:[EMAIL PROTECTED]
> Sent: Friday, January 11, 2008 12:27 PM
> To: [EMAIL PROTECTED]
> Cc: 'Jordan Crouse'; linuxbios@linuxbios.org
> Subject: Re: [LinuxBIOS] r3046 - trunk/LinuxBIOSv2/targets
> 
> Myles Watson wrote:
> >> Author: jcrouse
> >> Date: 2008-01-11 19:23:47 +0100 (Fri, 11 Jan 2008)
> >> New Revision: 3046
> >>
> >> Modified:
> >>    trunk/LinuxBIOSv2/targets/buildtarget
> >> Log:
> >> Add the ability to extend CFLAGS as needed for several new distros
> >> Signed-off-by: Ronald G. Minnich <[EMAIL PROTECTED]>
> >> Acked-by: Peter Stuge <[EMAIL PROTECTED]>
> >> Acked-by: Stefan Reinauer <[EMAIL PROTECTED]>
> >> Acked-by: Jordan Crouse <[EMAIL PROTECTED]>
> >>
> >>
> >>
> >> Modified: trunk/LinuxBIOSv2/targets/buildtarget
> >> ===================================================================
> >> --- trunk/LinuxBIOSv2/targets/buildtarget  2008-01-11 00:32:07 UTC
> (rev
> >> 3045)
> >> +++ trunk/LinuxBIOSv2/targets/buildtarget  2008-01-11 18:23:47 UTC
> (rev
> >> 3046)
> >> @@ -53,4 +53,25 @@
> >>  export PYTHONPATH=$config_dir
> >>  $PYTHON $config_py $config_lb $lbpath
> >>
> >> +# now start checking for distro-specific breakage.
> >> +## This check is for the no stack protector mess.
> >> +EXTRA_CFLAGS=
> >> +
> >> +if [ -z "$CC" ]; then
> >> +       CC=gcc
> >> +fi
> >> +
> >> +$CC -fno-stack-protector -S -xc /dev/null -o .$$.tmp
> >> +
> >> +if [ $? -eq 0 ]; then
> >> +       EXTRA_CFLAGS=-fno-stack-protector
> >> +fi
> >>
> >
> > This adds the flag for me, even though I didn't need it.  Was that
> intended?
> >
> > Myles
> I don't think it matters, if your gcc wasn't compiled to do stack
> checking it'll just ignore it. But I think it breaks gcc 3.x, do we
> care? IMHO, no.

In that case we shouldn't do a check, we should just add it to the flags.

Myles

> -Corey



-- 
linuxbios mailing list
linuxbios@linuxbios.org
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to