partially updating extra cmake

diff --git a/extra/CMakeLists.txt b/extra/CMakeLists.txt
index 9357abd67c49d192142e81ac4b8a1c10d58f319d..f17d9a3181e4e30ddb7c412574477c9789609609 100644 (file)
--- a/extra/CMakeLists.txt
+++ b/extra/CMakeLists.txt
@@ -12,21 +12,17 @@ else (BUILD_EXTRA_PLUGINS)
     set (CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_SOURCE_DIR}")
 
     # Look for Snort using pkg-config and our own find-snort module
-    find_package(PkgConfig REQUIRED)
-    pkg_check_modules(SNORT_PKG_MODULE snort)
 
     if (APPLE)
         set(CMAKE_MACOSX_RPATH OFF)
     endif(APPLE)
 
-    if (SNORT_PKG_MODULE_FOUND)
-        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${SNORT_PKG_MODULE_CFLAGS}")
-        find_package(SNORT QUIET)
-    else()
-        find_package(SNORT REQUIRED)
-    endif()
+    find_package(SNORT REQUIRED)
+
+
 
     if ((NOT SNORT_FOUND) AND (NOT SNORT_PKG_MODULE_FOUND))
+        find_package(SNORT REQUIRED) # to print the helpful Snort message
         message(FATAL_ERROR "Unable to find Snort includes!!")
     endif()
 

diff --git a/extra/FindSNORT.cmake b/extra/FindSNORT.cmake
index 99c17ca09b0f8705682bdd6f394278390c12b208..5f6ba8ad8c08a6752be90ca524f2eb85169c5a33 100644 (file)
--- a/extra/FindSNORT.cmake
+++ b/extra/FindSNORT.cmake
@@ -4,11 +4,9 @@
 #  This module defines
 #  DAQ_FOUND, if false, do not try to link to Lua
 # 
-#  SNORT_FOUND - system has the daq
-#  SNORT_INCLUDE_DIR - the daqs include directory
+#  SNORT_FOUND - Found Snort
+#  SNORT_INCLUDE_DIR - Snort include directory
 #
-## Copied from default CMake FindLua51.cmake
-
 
 set(ERROR_MESSAGE
     "
@@ -21,6 +19,9 @@ set(ERROR_MESSAGE
 
     3)  set the environment variable SNORT_DIR
 
+    4)  install pkg-config and add snort.pc to the PKG_CONFIG_PATH
+            environment variable.
+
     "
 )
 
@@ -28,9 +29,20 @@ set(ERROR_MESSAGE
 find_path (SNORT_INCLUDE_DIR
     NAMES main/snort_types.h
     HINTS ENV SNORT_DIR
-    PATH_SUFFIXES snort
 )
 
+if (NOT SNORT_INCLUDE_DIR)
+    find_package(PkgConfig QUIET)
+
+    if (PKG_CONFIG_FOUND)
+        pkg_check_modules(SNORT_PKG_MODULE snort)
+
+        if (SNORT_PKG_MODULE_FOUND)
+            set(SNORT_INCLUDE_DIR "${SNORT_PKG_MODULE_INCLUDE_DIRS}")
+        endif (SNORT_PKG_MODULE_FOUND)
+    endif (PKG_CONFIG_FOUND)
+endif (NOT SNORT_INCLUDE_DIR)
+
 
 include(FindPackageHandleStandardArgs)
 find_package_handle_standard_args(SNORT

diff --git a/extra/src/codecs/pim.cc b/extra/src/codecs/pim.cc
index b95f69ccba605b4832e514e7b0e6f62399bb69a3..0ec187b77714337cdb492543df373ad5e606306f 100644 (file)
--- a/extra/src/codecs/pim.cc
+++ b/extra/src/codecs/pim.cc
@@ -26,6 +26,7 @@
 #endif
 
 #include "framework/codec.h"
+#include "framework/decode_data.h"
 #include "codecs/codec_events.h"
 
 
@@ -57,7 +58,6 @@ void PimCodec::get_protocol_ids(std::vector<uint16_t>& v)
 {
     v.push_back(IPPROTO_ID_PIM);
 }
-
 bool PimCodec::decode(const RawData&, CodecData& codec, DecodeData&)
 {
     codec_events::decoder_event(codec, DECODE_IP_BAD_PROTO);

diff --git a/src/framework/codec.h b/src/framework/codec.h
index bf8a8a7c3eddc5cc9fd02be428cfa47c408c2632..4ddd7fcf79c8281948aba1b95d481bb5569d6532 100644 (file)
--- a/src/framework/codec.h
+++ b/src/framework/codec.h
@@ -407,5 +407,5 @@ struct CodecApi
     CdDelFunc dtor;   // clean up instance data
 };
 
-#endif
+#endif /* FRAMEWORK_CODEC_H */