From c7d86568a07845d56508baa88ece7853ce193eae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Valverde?= Date: Wed, 1 May 2019 19:02:34 +0100 Subject: [PATCH] CMake: Remove wsutil pcap dependency MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: Ic5a3653cb8bcc33e0be108c8b201567e7090f9f5 Reviewed-on: https://code.wireshark.org/review/33043 Petri-Dish: João Valverde Tested-by: Petri Dish Buildbot Reviewed-by: Peter Wu Reviewed-by: João Valverde --- capchild/CMakeLists.txt | 6 +++++- caputils/CMakeLists.txt | 1 + caputils/capture-pcap-util-unix.c | 2 +- caputils/capture-pcap-util.h | 2 +- caputils/capture_wpcap_packet.c | 2 +- extcap/androiddump.c | 2 +- ringbuffer.c | 2 +- tools/checkAPIs.pl | 4 ++-- trigcap.c | 2 +- ui/CMakeLists.txt | 2 +- ui/qt/capture_filter_syntax_worker.cpp | 2 +- ui/qt/compiled_filter_output.cpp | 2 +- wsutil/wspcap.h => wspcap.h | 0 wsutil/CMakeLists.txt | 1 - 14 files changed, 17 insertions(+), 13 deletions(-) rename wsutil/wspcap.h => wspcap.h (100%) diff --git a/capchild/CMakeLists.txt b/capchild/CMakeLists.txt index cfea58eb99..688cdbcbc9 100644 --- a/capchild/CMakeLists.txt +++ b/capchild/CMakeLists.txt @@ -26,7 +26,11 @@ add_library(capchild STATIC ${CAPCHILD_SRC} ) -target_link_libraries(capchild PRIVATE wsutil) +target_link_libraries(capchild + PRIVATE + wsutil + $<$:pcap::pcap> +) set_target_properties(capchild PROPERTIES LINK_FLAGS "${WS_LINK_FLAGS}" diff --git a/caputils/CMakeLists.txt b/caputils/CMakeLists.txt index ee9a7302b7..e345146e27 100644 --- a/caputils/CMakeLists.txt +++ b/caputils/CMakeLists.txt @@ -50,6 +50,7 @@ add_library(caputils STATIC target_link_libraries(caputils PUBLIC wsutil + $<$:pcap::pcap> PRIVATE ${NL_LIBRARIES} ) diff --git a/caputils/capture-pcap-util-unix.c b/caputils/capture-pcap-util-unix.c index a5a847a08c..384a9851bf 100644 --- a/caputils/capture-pcap-util-unix.c +++ b/caputils/capture-pcap-util-unix.c @@ -16,7 +16,7 @@ #ifdef HAVE_LIBPCAP -#include +#include "wspcap.h" #ifdef __APPLE__ #include diff --git a/caputils/capture-pcap-util.h b/caputils/capture-pcap-util.h index e0e3fa78e9..85750cff4d 100644 --- a/caputils/capture-pcap-util.h +++ b/caputils/capture-pcap-util.h @@ -17,7 +17,7 @@ extern "C" { #ifdef HAVE_LIBPCAP -#include +#include "wspcap.h" #include "capture_opts.h" diff --git a/caputils/capture_wpcap_packet.c b/caputils/capture_wpcap_packet.c index 754c2487fd..89cbf99643 100644 --- a/caputils/capture_wpcap_packet.c +++ b/caputils/capture_wpcap_packet.c @@ -18,7 +18,7 @@ #include #include -#include +#include "wspcap.h" /* XXX - yes, I know, I should move cppmagic.h to a generic location. */ #include "tools/lemon/cppmagic.h" diff --git a/extcap/androiddump.c b/extcap/androiddump.c index 8ca775d496..3f273a3411 100644 --- a/extcap/androiddump.c +++ b/extcap/androiddump.c @@ -54,7 +54,7 @@ #define PCAP_RECORD_HEADER_LENGTH 16 #ifdef ANDROIDDUMP_USE_LIBPCAP - #include + #include "wspcap.h" #include #include diff --git a/ringbuffer.c b/ringbuffer.c index 08b9fbe311..2069f4de98 100644 --- a/ringbuffer.c +++ b/ringbuffer.c @@ -37,7 +37,7 @@ #include #include -#include +#include "wspcap.h" #include diff --git a/tools/checkAPIs.pl b/tools/checkAPIs.pl index bd8c69e249..667bab1b00 100755 --- a/tools/checkAPIs.pl +++ b/tools/checkAPIs.pl @@ -530,14 +530,14 @@ sub check_included_files($$) } } - # only our wrapper file wsutils/wspcap.h may include pcap.h + # only our wrapper file wspcap.h may include pcap.h # all other files should include the wrapper if ($filename !~ /wspcap\.h/) { foreach (@incFiles) { if ( m#([<"]|/+)pcap\.h[>"]$# ) { print STDERR "Warning: ".$filename. " includes pcap.h directly. ". - "Include wsutil/wspcap.h instead.\n"; + "Include wspcap.h instead.\n"; last; } } diff --git a/trigcap.c b/trigcap.c index 9086b5897f..495a24d37f 100644 --- a/trigcap.c +++ b/trigcap.c @@ -15,7 +15,7 @@ #include #include -#include +#include "wspcap.h" static int dumping; static volatile int keep_going; diff --git a/ui/CMakeLists.txt b/ui/CMakeLists.txt index 48f08f26b3..f430bb079b 100644 --- a/ui/CMakeLists.txt +++ b/ui/CMakeLists.txt @@ -86,7 +86,7 @@ set_target_properties(ui PROPERTIES FOLDER "UI" ) -target_link_libraries(ui wsutil) +target_link_libraries(ui wsutil caputils) if (HTML_HELP_COMPILER) add_definitions(-DHHC_DIR) diff --git a/ui/qt/capture_filter_syntax_worker.cpp b/ui/qt/capture_filter_syntax_worker.cpp index 38a5f80756..03fc52ed76 100644 --- a/ui/qt/capture_filter_syntax_worker.cpp +++ b/ui/qt/capture_filter_syntax_worker.cpp @@ -12,7 +12,7 @@ #ifdef HAVE_LIBPCAP #include -#include +#include "wspcap.h" #include "capture_opts.h" #include "ui/capture_globals.h" diff --git a/ui/qt/compiled_filter_output.cpp b/ui/qt/compiled_filter_output.cpp index 632d5239d8..e1c413cd72 100644 --- a/ui/qt/compiled_filter_output.cpp +++ b/ui/qt/compiled_filter_output.cpp @@ -13,7 +13,7 @@ #include "compiled_filter_output.h" #ifdef HAVE_LIBPCAP -#include +#include "wspcap.h" #endif #include "capture_opts.h" diff --git a/wsutil/wspcap.h b/wspcap.h similarity index 100% rename from wsutil/wspcap.h rename to wspcap.h diff --git a/wsutil/CMakeLists.txt b/wsutil/CMakeLists.txt index 17c6949133..d5854dfac2 100644 --- a/wsutil/CMakeLists.txt +++ b/wsutil/CMakeLists.txt @@ -270,7 +270,6 @@ target_link_libraries(wsutil ${GCRYPT_LIBRARIES} ${WIN_WS2_32_LIBRARY} ${GNUTLS_LIBRARIES} - $<$:pcap::pcap> ) if(WIN32)