Re: [PATH] Get rid of very annoying header warnings on darwin
On Thursday, September 11, 2003, at 07:50 pm, Brent Dax wrote: Arthur Bergman: # This gets rid of the very annoying long double might change warning # under Darwin... Thanks, applied (config/init/hints/darwin.pl version 1.7). However, can you see if the diff below my sig (applied against 1.7, not 1.6) works too? If so, it's probably a more appropriate place to put it. Nope, that doesn't work, cc_warn munges the input and only removes things from the command line, not adds new ones. so -Wno-shadow gets turned into remove -Wshadow Arthur
Re: [PATH] Get rid of very annoying header warnings on darwin
On Fri, Sep 12, 2003 at 12:57:45PM +0100, Arthur Bergman wrote: On Thursday, September 11, 2003, at 07:50 pm, Brent Dax wrote: Arthur Bergman: # This gets rid of the very annoying long double might change warning # under Darwin... Thanks, applied (config/init/hints/darwin.pl version 1.7). However, can you see if the diff below my sig (applied against 1.7, not 1.6) works too? If so, it's probably a more appropriate place to put it. Nope, that doesn't work, cc_warn munges the input and only removes things from the command line, not adds new ones. so -Wno-shadow gets turned into remove -Wshadow Try now. In a fit of, er, bloody mindedness I've s/cc_warn/ccwarn/g across the config system. It was broken. In theory you could specify warnings flags on the Configure.pl command line, but in practise they got lost. Now that bit should all work, and I think that the -Wno-foo stuff in hints files also bites correctly. It seems to be quite cunning, in that -Wno-foo will eliminate -Wfoo if present, otherwise -Wno-foo stays in. Or at least my experiments seem to suggest this. I've no idea quite how it's working. I think it's still fractionally broken, in that -Wno-format will erase -Wformat-nonliteral Nicholas Clark
Re: [PATH] Get rid of very annoying header warnings on darwin
On Fri, Sep 12, 2003 at 10:54:27PM +0100, Nicholas Clark wrote: I think it's still fractionally broken, in that -Wno-format will erase -Wformat-nonliteral A suitable anchor seemed to fix that. Nicholas Clark
[PATH] Get rid of very annoying header warnings on darwin
Hi, This gets rid of the very annoying long double might change warning under darwin, the warning comes from my config.h which has typedef long double HUGEFLOATVAL; Arthur Index: config/init/hints/darwin.pl === RCS file: /cvs/public/parrot/config/init/hints/darwin.pl,v retrieving revision 1.6 diff -u -r1.6 darwin.pl --- config/init/hints/darwin.pl 7 Jun 2002 19:21:17 - 1.6 +++ config/init/hints/darwin.pl 11 Sep 2003 14:38:57 - @@ -1,6 +1,6 @@ my($ccflags, $ldflags)=Configure::Data-get(qw(ccflags ldflags)); -$ccflags .= -pipe -fno-common ; +$ccflags .= -pipe -fno-common -Wno-long-double ; $ccflags =~ s/-flat_namespace\s*//; $ldflags =~ s/-flat_namespace\s*//; $ldflags .= -flat_namespace ;
RE: [PATH] Get rid of very annoying header warnings on darwin
Arthur Bergman: # This gets rid of the very annoying long double might change warning # under Darwin... Thanks, applied (config/init/hints/darwin.pl version 1.7). However, can you see if the diff below my sig (applied against 1.7, not 1.6) works too? If so, it's probably a more appropriate place to put it. --Brent Dax [EMAIL PROTECTED] Perl and Parrot hacker Yeah, and my underwear is flame-retardant--that doesn't mean I'm gonna set myself on fire to prove it. Index: config/init/hints/darwin.pl === RCS file: /cvs/public/parrot/config/init/hints/darwin.pl,v retrieving revision 1.7 diff -u -r1.7 darwin.pl --- config/init/hints/darwin.pl 11 Sep 2003 18:43:41 - 1.7 +++ config/init/hints/darwin.pl 11 Sep 2003 18:46:43 - @@ -1,6 +1,6 @@ my($ccflags, $ldflags)=Configure::Data-get(qw(ccflags ldflags)); -$ccflags .= -pipe -fno-common -Wno-long-double ; +$ccflags .= -pipe -fno-common ; $ccflags =~ s/-flat_namespace\s*//; $ldflags =~ s/-flat_namespace\s*//; $ldflags .= -flat_namespace ; @@ -8,5 +8,5 @@ Configure::Data-set( ccflags = $ccflags, ldflags = $ldflags, - cc_warn = -Wno-shadow, + cc_warn = -Wno-shadow -Wno-long-double, );