kou commented on pull request #11375: URL: https://github.com/apache/arrow/pull/11375#issuecomment-940428465
> > We already use `boost::process` for S3 tests. So it's not a new dependency. > > Could you clarify what part of the change are you commenting on? Is it the changes to `vcpkg.json`? If so, while it is true that `boost::process` was used already, it was not declared as a vcpkg dependency and thus not built by default. Sure. This is an answer of https://github.com/apache/arrow/pull/11375#issuecomment-939558494 > boost process seems like a new dependency on boost? I don't know if there are any issue taking it (I guess this is only for tests)? . I wanted to say that we don't need to think about `boost::process` in this pull request. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
