On Jul 21, 2013, at 7:06 PM, Joerg Mayer <[email protected]> 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 <[email protected]> > 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 <[email protected]> > Archives: http://www.wireshark.org/lists/wireshark-dev > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev > mailto:[email protected]?subject=unsubscribe ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
