Dominique Pelle wrote:

> >> The build log is available at
> >> http://kojipkgs.fedoraproject.org/packages/vim/7.2.245/1.fc12/data/logs/i686/build.log
> >> and we were using -D_FORTIFY_SOURCE=2 to compile vim as you've already
> >> suspected.
> >>
> >> This compile option comes from the RPM compile options and seems to take
> >> precedence over your fortify changes from patch 44 as we now have
> >> '-D_FORTIFY_SOURCE=2 -D_FORTIFY_SOURCE=1' on the gcc commandline.
> >
> > Ah, so the configure script is working, but not as expected.
> >
> > Where is the -D_FORTIFY_SOURCE=2 flag coming from?  Is it in $CFLAGS?
> > If so then configure could filter it out.
> 
> 
> The attached patch does what you suggest.
> 
> I tested with:
> 
> $ export CFLAGS="-Wp,-D_FORTIFY_SOURCE=2 -D_FORTIFY_SOURCE=2"
> $ cd vim7/src
> $ make distclean
> $ autoconf
> $ ./configure
> $ make
> 
> ... and -D_FORTIFY_SOURCE=1 is used when compiling.

Thanks, I'll include this.

-- 
hundred-and-one symptoms of being an internet addict:
158. You get a tuner card so you can watch TV while surfing.

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Raspunde prin e-mail lui