Skip to content

Commit 7d11484

Browse files
authored
GH-48342: [R] Turn off gcs by default, also if it is on, bundle. (#48343)
### Rationale for this change Building GCS is fragile in places and we're not sure how many (if anyone) really needs GCS. So let's turn it off by default and get some signal that folks want it back (also hopefully by that time we're on a more modern ABSL) ### What changes are included in this PR? Turn off GCS by default in R builds, also bundle when it is enabled. ### Are these changes tested? ### Are there any user-facing changes? No more GCS by default * GitHub Issue: #48342
1 parent 0f51aba commit 7d11484

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

r/inst/build_arrow_static.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ ${CMAKE} -DARROW_BOOST_USE_SHARED=OFF \
7878
-DBoost_SOURCE=${Boost_SOURCE:-} \
7979
-Dlz4_SOURCE=${lz4_SOURCE:-} \
8080
-DARROW_FILESYSTEM=ON \
81-
-DARROW_GCS=${ARROW_GCS:-$ARROW_DEFAULT_PARAM} \
81+
-DARROW_GCS=${ARROW_GCS:-OFF} \
8282
-DARROW_JEMALLOC=${ARROW_JEMALLOC:-$ARROW_DEFAULT_PARAM} \
8383
-DARROW_MIMALLOC=${ARROW_MIMALLOC:-ON} \
8484
-DARROW_JSON=${ARROW_JSON:-ON} \
@@ -102,6 +102,7 @@ ${CMAKE} -DARROW_BOOST_USE_SHARED=OFF \
102102
-DCMAKE_UNITY_BUILD=${CMAKE_UNITY_BUILD:-OFF} \
103103
-DOPENSSL_ROOT_DIR=${OPENSSL_ROOT_DIR} \
104104
-Dre2_SOURCE=${re2_SOURCE:-BUNDLED} \
105+
-Dabsl_SOURCE=${absl_SOURCE:-BUNDLED} \
105106
-Dxsimd_SOURCE=${xsimd_SOURCE:-} \
106107
-Dzstd_SOURCE=${zstd_SOURCE:-} \
107108
${EXTRA_CMAKE_FLAGS} \

0 commit comments

Comments
 (0)