kevingurney commented on PR #37773: URL: https://github.com/apache/arrow/pull/37773#issuecomment-1724207859
Enabling building of the `ARROW_CSV` component caused some build failures due to `RapidJSON` being inaccessible when building integration tests: https://github.com/apache/arrow/actions/runs/6225733757/job/16896790846?pr=37773#step:9:681. We could try to fix these build failures by explicitly installing `RapidJSON` into the GitHub Actions environments. However, considering that we were [already considering removing `GoogleTest` support from the MATLAB build system](#37532) - it might be better to just use this opportunity to do that (this is what the [Python bindings did, as well](https://github.com/apache/arrow/pull/14117)). I have already confirmed that not building the Arrow C++ library tests when `ARROW_CSV` is enabled does work on all platforms in another CI job in `mathworks/arrow`: https://github.com/mathworks/arrow/actions/runs/6226385022 I'll start working on a separate PR to remove `GoogleTest` support from the CMake build system for the MATLAB interface. In the meantime, I will leave this PR open, and I will rebase once `GoogleTest` support has been removed. -- 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]
