raulcd opened a new pull request, #48092:
URL: https://github.com/apache/arrow/pull/48092

   ## Do not merge
   This PR is implemented on top of:
   - https://github.com/apache/arrow/pull/48075
   
   ### Rationale for this change
   
   As a follow up of requiring a minimum CMake version >= 3.25 we discussed 
moving our dependencies from ExternalProject to FetchContent. This can heavily 
simplify our third party dependency management. Moving c-ares is the next step 
before moving grpc.
   
   ### What changes are included in this PR?
   
   The general change is moving from `ExternalProject` to `FetchContent`. 
   
   It also add some required integration due to other dependencies, like grpc, 
using `ExternalProject`. We not only have to build but also install in order 
for those other dependencies to find c-ares. This causes some timing issues 
between config, build, install that requires us to create a custom target to 
depend on so the other dependencies find abseil.
   
   ### Are these changes tested?
   
   Yes, the changes are tested locally and on CI.
   
   ### Are there any user-facing changes?
   
   No


-- 
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]

Reply via email to