Re: [Wireshark-commits] rev 52649: /trunk/ /trunk/: configure.ac

[Gerald Combs <gerald () wireshark org>]

On 10/16/13 1:52 PM, Evan Huus wrote:
> On Wed, Oct 16, 2013 at 2:40 PM,  <gerald () wireshark org> 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.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe