Re: [PATH] Get rid of very annoying header warnings on darwin

2003-09-12 Thread Arthur Bergman
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

2003-09-12 Thread Nicholas Clark
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

2003-09-12 Thread Nicholas Clark
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


RE: [PATH] Get rid of very annoying header warnings on darwin

2003-09-11 Thread Brent Dax
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,
 );