Skip to content

Commit

Permalink
maybe better configure fix
Browse files Browse the repository at this point in the history
  • Loading branch information
paleolimbot committed Jul 12, 2023
1 parent 018ca98 commit fba6479
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions r/configure
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,10 @@
# Currently the configure script doesn't offer much to make this easy.
# If you expect to rebuild multiple times, you should set up a dev
# environment.
# * Installing a dev version as a regular developer.
# * Installing a dev version as a regular developer.
# The best way is to maintain your own cmake build and install it
# to a directory (not system) that you set as the env var
# $ARROW_HOME.
# $ARROW_HOME.
#
# For more information, see the various installation and developer vignettes.

Expand Down Expand Up @@ -177,7 +177,7 @@ find_arrow () {
else
PC_LIB_VERSION=`grep '^Version' ${_LIBARROW_FOUND}/lib/pkgconfig/arrow.pc | sed s/Version:\ //`
fi
# This is in an R script for convenience and testability.
# This is in an R script for convenience and testability.
# Success means the found C++ library is ok to use.
# Error means the versions don't line up and we shouldn't use it.
# More specific messaging to the user is in the R script
Expand Down Expand Up @@ -253,7 +253,6 @@ do_autobrew () {
cp tools/autobrew .
fi
fi

if ! . autobrew; then
echo "Failed to retrieve binary for ${PKG_BREW_NAME}"
fi
Expand Down

0 comments on commit fba6479

Please sign in to comment.