On 31 May 2013, at 10:05, Arran Cudbard-Bell <a.cudba...@freeradius.org> wrote:

> 
> On 31 May 2013, at 09:03, Alan DeKok <al...@deployingradius.com> wrote:
> 
>> Phil Mayers wrote:
>>> Looks like it's trying to use a clang argument with gcc. Probably
>>> related to commit 4cbe9552c
>> 
>> Yeah.  Clang shows up as "GCC" on configure's tests.  Arran didn't
>> check to see if that really was clang.
> 
> Because you shouldn't need to, it was intentional. With -Werror GCC should 
> warn (and therefore fail) if the argument isn't supported.

Ok pushed a better fix.

The issue is -Q has a different meaning in GCC.

       -Q  Makes the compiler print out each function name as it is compiled, 
and print some            statistics about each pass when it finishes.

Note that Q does not take additional parameters, yet GCC does not complain (in 
some versions) when we pass:

        -Qunused-arguments

The fix is:
        -Werror -Qunused-arguments -foobar

If -Qunused-arguments has the desired effect, -foobar won't generate a warning.

Arran Cudbard-Bell <a.cudba...@freeradius.org>
FreeRADIUS Development Team

-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to