nealrichardson commented on a change in pull request #8756:
URL: https://github.com/apache/arrow/pull/8756#discussion_r529726847
##########
File path: cpp/cmake_modules/DefineOptions.cmake
##########
@@ -363,6 +363,8 @@ if("${CMAKE_SOURCE_DIR}" STREQUAL
"${CMAKE_CURRENT_SOURCE_DIR}")
define_option(ARROW_WITH_UTF8PROC
"Build with support for Unicode properties using the utf8proc
library" ON)
+ define_option(ARROW_WITH_RE2
+ "Build with support for regular expressions using the re2
library" ON)
Review comment:
I find this confusing. I understand how it works--if ARROW_COMPUTE is
on, these are on too unless you set them to OFF--but it looks like they're
always built by default, unlike the compression libs listed above.
Maybe it's irrelevant that it looks confusing here. Or maybe at least we
should add a comment here explaining how the default is really ON iff
ARROW_COMPUTE is ON too. Or is it possible to define the default option here to
be literally whatever ARROW_COMPUTE is, like `define_option(ARROW_WITH_RE2
"..." ARROW_COMPUTE)`?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
