kou commented on PR #44005:
URL: https://github.com/apache/arrow/pull/44005#issuecomment-2357506325
We don't need to change `cpp/src/arrow/testing/process.cc` because it uses
`boost::filesystem` only for `boost::process`.
Could you update the PR description?
We can simplify CMake codes a bit:
```diff
diff --git a/cpp/src/arrow/io/CMakeLists.txt
b/cpp/src/arrow/io/CMakeLists.txt
index f7afbca558..0398422b05 100644
--- a/cpp/src/arrow/io/CMakeLists.txt
+++ b/cpp/src/arrow/io/CMakeLists.txt
@@ -28,9 +28,7 @@ if(ARROW_HDFS)
PREFIX
"arrow-io"
EXTRA_LINK_LIBS
- arrow::hadoop
- Boost::filesystem
- Boost::system)
+ arrow::hadoop)
endif()
add_arrow_test(memory_test PREFIX "arrow-io")
```
If we can remove boost::process dependency, we can remove more codes...
--
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
