Re: [LinuxBIOS] r3046 - trunk/LinuxBIOSv2/targets

[Myles Watson]

> 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 
> Acked-by: Peter Stuge 
> Acked-by: Stefan Reinauer 
> Acked-by: Jordan Crouse
> 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
> +rm -rf .$$.tmp
>
> +for i in $build_dir/Makefile.settings $build_dir/*/Makefile.settings
> +do
> echo CFLAGS+=$EXTRA_CFLAGS >>$i
>
> +done
>
> exit $?