nealrichardson commented on a change in pull request #10710:
URL: https://github.com/apache/arrow/pull/10710#discussion_r678431911
##########
File path: r/configure
##########
@@ -80,7 +80,8 @@ elif [ "$INCLUDE_DIR" ] && [ "$LIB_DIR" ]; then
else
# Use pkg-config if available and allowed
pkg-config --version >/dev/null 2>&1
- if [ $? -eq 0 ] && [ "$ARROW_USE_PKG_CONFIG" != "false" ]; then
+ if [ $? -eq 0 ] && [ "$ARROW_USE_PKG_CONFIG" != "false" ] && [
"$ARROW_DEPENDENCY_SOURCE" == "AUTO" ]; then
Review comment:
However, you may want to pull Kou's suggestion from
https://github.com/apache/arrow/pull/10404#issuecomment-871034191 in here
(though I'm not certain about the `--static` part in this block, may have to
experiment locally)
--
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]
