Re: [PATCH] probe for gcc -Wxxx only when gcc (well, g++)

2008-01-09 Thread Andy Lester
Unfortunately I haven't been able to test the patch, however, icc *should* handle -W flags exactly the same as gcc. And if it doesn't, then there is an issue there we (or Intel) should deal with. So, I would update the patch to ask if we have gcc or icc. I agree that -W doesn't apply to some ot

Re: [PATCH] probe for gcc -Wxxx only when gcc (well, g++)

2008-01-09 Thread Paul Cochrane
On 09/01/2008, chromatic <[EMAIL PROTECTED]> wrote: > On Monday 07 January 2008 20:09:26 Jarkko Hietaniemi wrote: > > > --- config/auto/warnings.pm.dist 2008-01-08 05:51:42.0 +0200 > > +++ config/auto/warnings.pm 2008-01-08 06:01:23.0 +0200 > > @@ -132,17 +132,22 @@ > >

Re: [PATCH] probe for gcc -Wxxx only when gcc (well, g++)

2008-01-08 Thread chromatic
On Monday 07 January 2008 20:09:26 Jarkko Hietaniemi wrote: > --- config/auto/warnings.pm.dist 2008-01-08 05:51:42.0 +0200 > +++ config/auto/warnings.pm 2008-01-08 06:01:23.0 +0200 > @@ -132,17 +132,22 @@ > $verbose = $conf->options->get('verbose'); > print "\n" if

Re: [PATCH] probe for gcc -Wxxx only when gcc (well, g++)

2008-01-08 Thread James E Keenan
Jarkko Hietaniemi wrote: --- config/auto/warnings.pm.dist2008-01-08 05:51:42.0 +0200 +++ config/auto/warnings.pm 2008-01-08 06:01:23.0 +0200 @@ -132,17 +132,22 @@ Thanks, Jarkko. Since I'm working on correcting other problems with auto::warnings and its tests in an

[PATCH] probe for gcc -Wxxx only when gcc (well, g++)

2008-01-08 Thread Jarkko Hietaniemi
--- config/auto/warnings.pm.dist2008-01-08 05:51:42.0 +0200 +++ config/auto/warnings.pm 2008-01-08 06:01:23.0 +0200 @@ -132,17 +132,22 @@ $verbose = $conf->options->get('verbose'); print "\n" if $verbose; -# add on some extra warnings if requested -pu