kou commented on PR #36551:
URL: https://github.com/apache/arrow/pull/36551#issuecomment-1636430729

   The warning from Google Cloud C++: We will be able to suppress it by using 
`-DOPENSSL_ROOT_DIR` instead of `-DOPENSSL_*` here:
   
   
https://github.com/apache/arrow/blob/d8982fef34dcc6cda58ff2051804932f7bd39c37/cpp/cmake_modules/ThirdpartyToolchain.cmake#L4140-L4142
   
   But I think that the warning is false-positive because we specify OpenSSL 
information explicitly. So I think that we can choose 
`-DGOOGLE_CLOUD_CPP_ENABLE_MACOS_OPENSSL_CHECK=OFF` instead of using 
`-DOPENSSL_ROOT_DIR`.
   
   > I think we (on purpose?) link to system curl but use OpenSSL from autobrew.
   
   This may be the cause.
   We may be able to solve this problem by using curl from autobrew too.
   
   > I am going to re-run tests/packaging and verification on the maintenance 
branch if only the autobrew / brew / r-binary jobs fail (+ the known spark 
ones) I think we can move forward and treat this issue as we usually do with 
other CRAN related issues, making minor modifications after the release to get 
R released on CRAN.
   
   +1
   


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