Skip to content

Commit

Permalink
Merge pull request #18411 from dcamper/hpcc-31366-use-pcre2-not-boost
Browse files Browse the repository at this point in the history
HPCC-31366 Replace Boost regex with PCRE2 regex

Reviewed-by: Gordon Smith <[email protected]>
Reviewed-by: Gavin Halliday <[email protected]>
Merged-by: Gavin Halliday <[email protected]>
  • Loading branch information
ghalliday authored Apr 8, 2024
2 parents 8998b32 + 37501ab commit 3b808d4
Show file tree
Hide file tree
Showing 9 changed files with 1,195 additions and 103 deletions.
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,4 @@ eclcc.log
./dockerfiles/platform-build-incremental/hpcc.gitpatch
dockerfiles/platform-build-incremental/hpcc.gitpatch
.env
/vcpkg.json
/vcpkg.json
5 changes: 0 additions & 5 deletions cmake_modules/commonSetup.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -839,11 +839,6 @@ IF ("${COMMONSETUP_DONE}" STREQUAL "")
endif()
endif(USE_URIPARSER)

find_package(Boost COMPONENTS regex)
if (NOT Boost_REGEX_FOUND)
message(FATAL_ERROR "BOOST_REGEX required but package not found")
endif()

if(USE_PARQUET)
message(STATUS "Parquet enabled")
add_definitions(-D_USE_PARQUET)
Expand Down
1 change: 0 additions & 1 deletion cmake_modules/options.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ option(SIGN_MODULES "Enable signing of ecl standard library modules" OFF)
option(USE_CPPUNIT "Enable unit tests (requires cppunit)" OFF)
option(USE_OPENLDAP "Enable OpenLDAP support (requires OpenLDAP)" ON)
option(USE_ICU "Enable unicode support (requires ICU)" ON)
option(Boost_USE_STATIC_LIBS "Use boost_regex static library for RPM BUILD" OFF)
option(USE_OPENSSL "Configure use of OpenSSL" ON)
option(USE_OPENSSLV3 "Configure use of OpenSSL Version 3 or newer" ON)
option(USE_ZLIB "Configure use of zlib" ON)
Expand Down
4 changes: 2 additions & 2 deletions rtl/eclrtl/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ include_directories (
./../../system/jlib
./../../roxie/roxiemem
./../../testing/unittests
${Boost_INCLUDE_DIRS}
)

ADD_DEFINITIONS( -D_USRDLL -DECLRTL_EXPORTS )
Expand All @@ -87,7 +86,8 @@ endif ()

HPCC_ADD_LIBRARY( eclrtl SHARED ${SRCS} )

target_link_libraries( eclrtl Boost::boost Boost::regex )
find_package(pcre2 CONFIG REQUIRED)
target_link_libraries( eclrtl PCRE2::8BIT PCRE2::16BIT )

target_link_libraries ( eclrtl
jlib
Expand Down
Loading

0 comments on commit 3b808d4

Please sign in to comment.