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

   ### Rationale for this change
   
   We need "Requires.prviate: libcurl openssl" in arrow.pc only when we use 
static AWS SDK for C++ (bundled AWS SDK for C++).
   
   In general, "Requires.private: libcurl openssl" is harmless with system 
(shared) AWS SDK for C++. But "openssl" is harmful on macOS with Homebrew 
because OpenSSL from Homebrew isn't installed to `$(brew --prefix)`. It means 
that users who want to use OpenSSL from Homebrew need to refer OpenSSL by 
`$(brew --prefix [email protected])` or `$(brew --prefix [email protected])` explicitly.
   
   So it's helpful that we don't add needless "Requires.private: libcurl 
openssl" to arrow.pc.
   
   ### What changes are included in this PR?
   
   These changes stop adding `Requires.private: libcurl openssl` when we use 
system AWS SDK for C++.
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   Yes.


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