nealrichardson opened a new pull request #8243:
URL: https://github.com/apache/arrow/pull/8243


   I've tried enabling this in the R linux builds and have made some progress, 
but I'm hitting some issues that someone more experienced with cmake might be 
able to help with.
   
   * In the everything bundled static build that is the default for R on Linux 
(the rstudio/r-base centos jobs), the addition of aws-sdk-cpp means that 
openssl is required, but the system doesn't have it, and we don't have a 
bundled openssl external project. TileDB does: 
https://github.com/TileDB-Inc/TileDB/blob/dev/cmake/Modules/FindOpenSSL_EP.cmake#L85-L118
 so maybe we can use that for inspiration.
   * In the ubuntu R jobs that are based on the ubuntu-cpp image, openssl is 
present so the build doesn't fail, but with ARROW_DEPENDENCY_SOURCE=AUTO, cmake 
fails unless you set AWSSDK_SOURCE=BUNDLED because it can't "find" it. We 
should fix that.
   * Setting AWSSDK_SOURCE=BUNDLED, [awssdk_ep builds 
successfully](https://github.com/nealrichardson/arrow/runs/1151722752?check_suite_focus=true#step:8:1379),
 but Arrow s3fs.cc compilation fails because it cant find the aws-sdk headers. 
Looking at build_awssdk and comparing with other ep's, it's surely incomplete, 
and among other things it probably needs `-DBUILD_SHARED_LIBS=OFF`. This is 
where I'm not sure how to finish wiring this ep up. Once again, TileDB may be a 
point of reference: 
https://github.com/TileDB-Inc/TileDB/blob/dev/cmake/Modules/FindAWSSDK_EP.cmake
   
   Solving this will also let us enable S3 in more Python wheels; cf. #7997


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to