From f13cbd692ee658a6e619e41b115bb2aaee7f5eda Mon Sep 17 00:00:00 2001 From: "Chang, Harry" Date: Thu, 20 Dec 2018 15:28:07 +0800 Subject: [PATCH] Update PCRE version to 8.42 (8.41 is also compatible) --- CMakeLists.txt | 4 ++-- cmake/pcre.cmake | 12 ++++++------ doc/dev-reference/compilation.rst | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 07000270..0e905db6 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -456,7 +456,7 @@ set(PCRE_REQUIRED_MINOR_VERSION 41) set(PCRE_REQUIRED_VERSION ${PCRE_REQUIRED_MAJOR_VERSION}.${PCRE_REQUIRED_MINOR_VERSION}) include (${CMAKE_MODULE_PATH}/pcre.cmake) if (NOT CORRECT_PCRE_VERSION) - message(STATUS "PCRE ${PCRE_REQUIRED_VERSION} not found") + message(STATUS "PCRE ${PCRE_REQUIRED_VERSION} or above not found") endif() # we need static libs for Chimera - too much deep magic for shared libs @@ -508,7 +508,7 @@ set(PCRE_REQUIRED_MINOR_VERSION 41) set(PCRE_REQUIRED_VERSION ${PCRE_REQUIRED_MAJOR_VERSION}.${PCRE_REQUIRED_MINOR_VERSION}) include (${CMAKE_MODULE_PATH}/pcre.cmake) if (NOT CORRECT_PCRE_VERSION) - message(STATUS "PCRE ${PCRE_REQUIRED_VERSION} not found") + message(STATUS "PCRE ${PCRE_REQUIRED_VERSION} or above not found") endif() # we need static libs for Chimera - too much deep magic for shared libs diff --git a/cmake/pcre.cmake b/cmake/pcre.cmake index 2b0d23c7..e0acda5e 100644 --- a/cmake/pcre.cmake +++ b/cmake/pcre.cmake @@ -27,7 +27,7 @@ if (PCRE_BUILD_SOURCE) # first, check version number CHECK_C_SOURCE_COMPILES("#include - #if PCRE_MAJOR != ${PCRE_REQUIRED_MAJOR_VERSION} || PCRE_MINOR != ${PCRE_REQUIRED_MINOR_VERSION} + #if PCRE_MAJOR != ${PCRE_REQUIRED_MAJOR_VERSION} || PCRE_MINOR < ${PCRE_REQUIRED_MINOR_VERSION} #error Incorrect pcre version #endif main() {}" CORRECT_PCRE_VERSION) @@ -35,10 +35,10 @@ if (PCRE_BUILD_SOURCE) if (NOT CORRECT_PCRE_VERSION) unset(CORRECT_PCRE_VERSION CACHE) - message(STATUS "Incorrect version of pcre - version ${PCRE_REQUIRED_VERSION} is required") + message(STATUS "Incorrect version of pcre - version ${PCRE_REQUIRED_VERSION} or above is required") return () else() - message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION} - building from source.") + message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION} or above - building from source.") endif() # PCRE compile options @@ -52,12 +52,12 @@ if (PCRE_BUILD_SOURCE) else () # pkgconf should save us find_package(PkgConfig) - pkg_check_modules(PCRE libpcre=${PCRE_REQUIRED_VERSION}) + pkg_check_modules(PCRE libpcre>=${PCRE_REQUIRED_VERSION}) if (PCRE_FOUND) set(CORRECT_PCRE_VERSION TRUE) - message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION}") + message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION} or above") else () - message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION} not found") + message(STATUS "PCRE version ${PCRE_REQUIRED_VERSION} or above not found") return () endif () endif (PCRE_BUILD_SOURCE) diff --git a/doc/dev-reference/compilation.rst b/doc/dev-reference/compilation.rst index 7a7f37ec..214f4abc 100644 --- a/doc/dev-reference/compilation.rst +++ b/doc/dev-reference/compilation.rst @@ -64,7 +64,7 @@ libpcre are supported. The use of unsupported constructs will result in compilation errors. The version of PCRE used to validate Hyperscan's interpretation of this syntax -is 8.41. +is 8.41 or above. ==================== Supported Constructs