Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] rev 50749: /trunk/ /trunk/: CMakeLists.txt

From: Michael Tuexen <Michael.Tuexen@xxxxxxxxxxxxxxxxx>
Date: Sun, 21 Jul 2013 19:49:01 +0200
On Jul 21, 2013, at 7:06 PM, Joerg Mayer <jmayer@xxxxxxxxx> wrote:

> Hello Michael,
> 
> On Sun, Jul 21, 2013 at 06:45:11PM +0200, Michael Tuexen wrote:
>> [mba:wireshark/trunk/build] tuexen% clang --version
>> Apple LLVM version 4.2 (clang-425.0.28) (based on LLVM 3.2svn)
>> Target: x86_64-apple-darwin12.4.0
> 
> On my system:
> Lilly:svn jmayer$ clang --version
> Apple clang version 1.7 (tags/Apple/clang-77) (based on LLVM 2.9svn)
> Target: x86_64-apple-darwin10
> 
>> -- Performing Test WS_C_FLAG_VALID0
>> -- Performing Test WS_C_FLAG_VALID0 - Success
> ...
>> -- Performing Test WS_C_FLAG_VALID25 - Success
>> -- C-Flags:  -O2 -g -Wall -W -Wextra -Wendif-labels -Wpointer-arith -Warray-bounds -Wformat-security -Wshorten-64-to-32 -Wvla -Waddress -Warray-bounds -Wattributes -Wdiv-by-zero -Wignored-qualifiers -Wno-overlength-strings -Wwrite-strings -Wno-long-long -Wc++-compat -Wdeclaration-after-statement -Wshadow -Wno-pointer-sign -Wold-style-definition -Wshadow -Wstrict-prototypes
>> CXX-Flags:  -O2 -g -Wall -W -Wextra -Wendif-labels -Wpointer-arith -Warray-bounds -Wformat-security -Wshorten-64-to-32 -Wvla -Waddress -Warray-bounds -Wattributes -Wdiv-by-zero -Wignored-qualifiers -Wno-overlength-strings -Wwrite-strings -Wno-long-long
> 
> This matches with Guy's analysis although it doesn't show that behaviour on my
> system (which is too old no doubt). I will add code to detect that flag and if it
> works with gcc and older clang, then I will commit it and hope it will fix the
> detection problem on systems with newer clang.
> 
>> /* _U_ isn't needed for C++, simply don't name the variable.
>>   However, we do need it for some headers that are shared between C and C++. */
>> #define _U_ 
> 
>> So: It doesn't work. Is there a way I can print the value of CMAKE_C_COMPILER?
> 
> Index: CMakeLists.txt
> ===================================================================
> --- CMakeLists.txt      (revision 50757)
> +++ CMakeLists.txt      (working copy)
> @@ -189,6 +189,8 @@
>        -DG_DISABLE_SINGLE_INCLUDES
> )
> 
> +message(STATUS "C-Compiler: ${CMAKE_C_COMPILER}")
> +
> # Counterhack to work around some cache magic in CHECK_C_SOURCE_COMPILES
> include(CheckCCompilerFlag)
> set(C 0)
It reports:
C-Compiler: /usr/bin/cc

That explains why it doesn't detect it as clang...
> 
> Thanks for testing
.. not only testing. I'm using it. And it is bad to have all the
unused warnings, since then I don't detect bugs I would commit...

Thanks a lot for the cmake support. I really use it since Apple
doesn't install autotools anymore.

Best regards
Michael
>    Jörg
> 
> -- 
> Joerg Mayer                                           <jmayer@xxxxxxxxx>
> We are stuck with technology when what we really want is just stuff that
> works. Some say that should read Microsoft instead of technology.
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe