Re: pkg/52390: current fails to bootstrap on CentOS 6

[Jason Bacon <bacon4000%gmail.com@localhost>]

On 07/17/17 10:09, Joerg Sonnenberger wrote:
> On Mon, Jul 17, 2017 at 08:45:21AM -0500, Jason Bacon wrote:
> > > -Wno-unused-result, not -Wno-error=unused-result.
> > Also not supported in 4.4.7.
> F**k: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25509
>
> OK, I guess that makes it "disable fortify for GCC 4.4 on Linux". I'm
> not willing to disable all warnings just they couldn't get their act
> together.
>
> Joerg

Or disable -Werror for GCC 4.4 on Linux.  This seems to be sufficient 
based on my tests.

I'm not sure exactly how much less invasive it is than disabling 
fortify.  At least warnings from fortify are still shown and if we're 
using _FORTIFY_SOURCE=2, runtime errors can still be caught.

Linux sciprog2.ceas bacon ~ 405: (pkgsrc): gcc -O2 -D_FORTIFY_SOURCE=2 
warn.c
warn.c: In function main:
warn.c:9: warning: ignoring return value of fwrite, declared with 
attribute warn_unused_result
At top level:
cc1: warning: unrecognized command line option "-Wno-unused-result"
Linux sciprog2.ceas bacon ~ 406: (pkgsrc): ./a.out
Hello, world!

Linux sciprog2.ceas bacon ~ 410: (pkgsrc): gcc -O2 -D_FORTIFY_SOURCE=2 
-Werror warn.c
cc1: warnings being treated as errors
warn.c: In function ���main���:
warn.c:9: error: ignoring return value of ���fwrite���, declared with 
attribute warn_unused_result
Linux sciprog2.ceas bacon ~ 411: (pkgsrc): ./a.out
./a.out: Command not found.

Thanks,

    JB

--
Earth is a beta site.