kou commented on code in PR #45768:
URL: https://github.com/apache/arrow/pull/45768#discussion_r1994565051
##########
cpp/src/arrow/meson.build:
##########
@@ -371,25 +371,38 @@ install_headers(
)
if needs_tests
- filesystem_dep = dependency(
+ boost_dep = dependency(
'boost',
modules: ['filesystem'],
Review Comment:
In CMake, we can use separated CMake targets for Boost. For example,
`Boost::filesystem` and `Boost::process`. Can we do this with Meson?
Some libraries (e.g. Gandiva) need only `Boost::headers` but other libraries
(e.g. `arrow_testing`) need `Boost::headers` and `Boost::process`.
I don't want to link needless libraries as much as possible. (e.g. I don't
want to link `Boost::process` to Gandiva.)
--
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.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
