Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 52649: /trunk/ /trunk/: configure.ac
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Gerald Combs <gerald@xxxxxxxxxxxxx>
Date: Wed, 16 Oct 2013 15:42:54 -0700
On 10/16/13 1:52 PM, Evan Huus wrote:
> On Wed, Oct 16, 2013 at 2:40 PM,  <gerald@xxxxxxxxxxxxx> wrote:
>> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=52649
>>
>> User: gerald
>> Date: 2013/10/16 06:40 PM
>>
>> Log:
>>  Don't bother checking for clang, just add -Qunused-arguments to the
>>  compiler flags checks.
>>
>> Directory: /trunk/
>>   Changes    Path            Action
>>   +1 -3      configure.ac    Modified
> 
> This is causing weird configure errors for me, I suspect because we do
> some hackery in AC_WIRESHARK_COMPILER_FLAGS_CHECK that isn't being
> invoked because the flag doesn't start with a -W. Dunno if adding -Q
> to that list of prefixes is sufficient or if there is more at work...`

I added explicit tests for Clang in r52654. They update CFLAGS and
CXXFLAGS directly instead of relying on AC_WIRESHARK_COMPILER_FLAGS_CHECK.