Wireshark-commits: [Wireshark-commits] master 20accf3: Don't do the check for clang/clang++.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 18 Jun 2014 04:58:44 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=20accf341cef4f90997369c6aa26f748e3b9be76
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

20accf3 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't do the check for clang/clang++.
    
    Adding -Qunused-arguments to CXXFLAGS causes the checks for -f and -m
    flags not to fail with clang++, causing the configure script to warn
    about -f flags supported by clang but not clang++ indicating that the
    compilers are a mismatched pair.
    
    The checks we do for flags should eliminate "unused" -f/-m flags,
    suppressing the warnings that way.
    
    Change-Id: I749d6f499a3d34300518cc0ba539f355377359af
    Reviewed-on: https://code.wireshark.org/review/2362
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  4cebb06   Add -Werror when testing -m compiler flags.
    adds  20accf3   Don't do the check for clang/clang++.


Summary of changes:
 acinclude.m4 |   42 ------------------------------------------
 configure.ac |    1 -
 2 files changed, 43 deletions(-)