From 9481ba1b6e8aab8d9ddfd0b03672fb661b0c4fef Mon Sep 17 00:00:00 2001
From: Hiroyuki Sato <hiroysato@gmail.com>
Date: Mon, 3 Aug 2026 17:13:46 +0900
Subject: [PATCH 1/2] GH-50773: [CI][Dev] Fix shellcheck errors in the
 ci/scripts/r_sanitize.sh

---
 .pre-commit-config.yaml  |  1 +
 ci/scripts/r_sanitize.sh | 13 +++++++------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index a7512fd5bd3f..4d199e3f6f31 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -342,6 +342,7 @@ repos:
           ?^ci/scripts/r_deps\.sh$|
           ?^ci/scripts/r_docker_configure\.sh$|
           ?^ci/scripts/r_install_system_dependencies\.sh$|
+          ?^ci/scripts/r_install_system_dependencies\.sh$|
           ?^ci/scripts/r_revdepcheck\.sh$|
           ?^ci/scripts/release_test\.sh$|
           ?^ci/scripts/ruby_test\.sh$|
diff --git a/ci/scripts/r_sanitize.sh b/ci/scripts/r_sanitize.sh
index b66724fdbdc9..630517308ed3 100755
--- a/ci/scripts/r_sanitize.sh
+++ b/ci/scripts/r_sanitize.sh
@@ -18,12 +18,12 @@
 
 set -ex
 
-: ${R_BIN:=RDsan}
+: "${R_BIN:=RDsan}"
 
 source_dir=${1}/r
 rhome=$(${R_BIN} RHOME)
 
-pushd ${source_dir}
+pushd "${source_dir}"
 
 # Unity builds were causing the CI job to run out of memory
 export CMAKE_UNITY_BUILD=OFF
@@ -33,10 +33,10 @@ export ARROW_R_DEV=TRUE
 export CMAKE_BUILD_TYPE=RelWithDebInfo
 
 ncores=$(${R_BIN} -s -e 'cat(parallel::detectCores())')
-echo "MAKEFLAGS=-j${ncores}" >> ${rhome}/etc/Renviron.site
+echo "MAKEFLAGS=-j${ncores}" >> "${rhome}/etc/Renviron.site"
 
 # build first so that any stray compiled files in r/src are ignored
-${R_BIN} CMD build --no-build-vignettes --no-manual .
+"${R_BIN}" CMD build --no-build-vignettes --no-manual .
 
 # But unset the env var so that it doesn't cause us to run extra dev tests
 unset ARROW_R_DEV
@@ -47,12 +47,13 @@ export ARROW_R_VERBOSE_TEST=TRUE
 # We prune dependencies for these, so we need to disable forcing suggests
 export _R_CHECK_FORCE_SUGGESTS_=FALSE
 
-export SUPPRESSION_FILE=$(readlink -f "tools/ubsan.supp")
+SUPPRESSION_FILE="$(readlink -f tools/ubsan.supp)"
+export SUPPRESSION_FILE
 export UBSAN_OPTIONS="print_stacktrace=1,suppressions=${SUPPRESSION_FILE}"
 # From the old rhub image https://github.com/r-hub/rhub-linux-builders/blob/master/fedora-clang-devel-san/Dockerfile
 export ASAN_OPTIONS="alloc_dealloc_mismatch=0:detect_leaks=0:detect_odr_violation=0"
 
-${R_BIN} CMD check --no-manual --no-vignettes --no-build-vignettes arrow*.tar.gz
+"${R_BIN}" CMD check --no-manual --no-vignettes --no-build-vignettes arrow*.tar.gz
 
 # Find sanitizer issues, print the file(s) they are part of, and fail the job
 find . -type f -name "*Rout" -exec grep -l "runtime error\|SUMMARY: UndefinedBehaviorSanitizer" {} \; > sanitizer_errors.txt

From 4bf7da13ab9b1b06f9e72a8c5dad88d5a85472ac Mon Sep 17 00:00:00 2001
From: Hiroyuki Sato <hiroysato@gmail.com>
Date: Mon, 3 Aug 2026 21:36:25 +0900
Subject: [PATCH 2/2] GH-50773: [CI][Dev] Fix shellcheck errors in the
 ci/scripts/r_sanitize.sh

---
 .pre-commit-config.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 4d199e3f6f31..d22e790230f2 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -342,8 +342,8 @@ repos:
           ?^ci/scripts/r_deps\.sh$|
           ?^ci/scripts/r_docker_configure\.sh$|
           ?^ci/scripts/r_install_system_dependencies\.sh$|
-          ?^ci/scripts/r_install_system_dependencies\.sh$|
           ?^ci/scripts/r_revdepcheck\.sh$|
+          ?^ci/scripts/r_sanitize\.sh$|
           ?^ci/scripts/release_test\.sh$|
           ?^ci/scripts/ruby_test\.sh$|
           ?^ci/scripts/rust_build\.sh$|