From 2e4daa41ccee9b519d8eacbd36e0e713b403be84 Mon Sep 17 00:00:00 2001 From: Hiroyuki Sato Date: Fri, 22 Aug 2025 10:16:35 +0900 Subject: [PATCH 1/3] GH-47402: [CI][Dev] Fix shellcheck errors in the ci/scripts/python_test_emscripten.sh --- .pre-commit-config.yaml | 1 + ci/scripts/python_test_emscripten.sh | 7 ++++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index f545f5c280a..fe251be693a 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -342,6 +342,7 @@ repos: ?^ci/scripts/python_build\.sh$| ?^ci/scripts/python_sdist_build\.sh$| ?^ci/scripts/python_sdist_test\.sh$| + ?^ci/scripts/python_test_emscripten\.sh$| ?^ci/scripts/python_wheel_unix_test\.sh$| ?^ci/scripts/r_build\.sh$| ?^ci/scripts/r_revdepcheck\.sh$| diff --git a/ci/scripts/python_test_emscripten.sh b/ci/scripts/python_test_emscripten.sh index 4029722568b..621ac4bd0d1 100755 --- a/ci/scripts/python_test_emscripten.sh +++ b/ci/scripts/python_test_emscripten.sh @@ -25,14 +25,15 @@ set -ex build_dir=${1}/python pyodide_dist_dir=${2} -cd ${build_dir} +cd "${build_dir}" # note: this uses the newest wheel in dist +# shellcheck disable=SC2012 pyodide_wheel=$(ls -t dist/pyarrow*.whl | head -1) echo "-------------- Running emscripten tests in Node ----------------------" -python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=node +python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=node echo "-------------- Running emscripten tests in Chrome --------------------" -python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=chrome +python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=chrome From 46d28576bf32e96d5971ca2d471c5efcb5f280bf Mon Sep 17 00:00:00 2001 From: Hiroyuki Sato Date: Tue, 26 Aug 2025 15:48:52 +0900 Subject: [PATCH 2/3] Revert "GH-47402: [CI][Dev] Fix shellcheck errors in the ci/scripts/python_test_emscripten.sh" This reverts commit 2e4daa41ccee9b519d8eacbd36e0e713b403be84. --- .pre-commit-config.yaml | 1 - ci/scripts/python_test_emscripten.sh | 7 +++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index fe251be693a..f545f5c280a 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -342,7 +342,6 @@ repos: ?^ci/scripts/python_build\.sh$| ?^ci/scripts/python_sdist_build\.sh$| ?^ci/scripts/python_sdist_test\.sh$| - ?^ci/scripts/python_test_emscripten\.sh$| ?^ci/scripts/python_wheel_unix_test\.sh$| ?^ci/scripts/r_build\.sh$| ?^ci/scripts/r_revdepcheck\.sh$| diff --git a/ci/scripts/python_test_emscripten.sh b/ci/scripts/python_test_emscripten.sh index 621ac4bd0d1..4029722568b 100755 --- a/ci/scripts/python_test_emscripten.sh +++ b/ci/scripts/python_test_emscripten.sh @@ -25,15 +25,14 @@ set -ex build_dir=${1}/python pyodide_dist_dir=${2} -cd "${build_dir}" +cd ${build_dir} # note: this uses the newest wheel in dist -# shellcheck disable=SC2012 pyodide_wheel=$(ls -t dist/pyarrow*.whl | head -1) echo "-------------- Running emscripten tests in Node ----------------------" -python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=node +python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=node echo "-------------- Running emscripten tests in Chrome --------------------" -python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=chrome +python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=chrome From e1ecc13c5fec251ce17541f4f5ddd5d9a8398fc4 Mon Sep 17 00:00:00 2001 From: Hiroyuki Sato Date: Fri, 22 Aug 2025 10:16:35 +0900 Subject: [PATCH 3/3] GH-47402: [CI][Dev] Fix shellcheck errors in the ci/scripts/python_test_emscripten.sh --- .pre-commit-config.yaml | 1 + ci/scripts/python_test_emscripten.sh | 7 ++++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index f545f5c280a..fe251be693a 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -342,6 +342,7 @@ repos: ?^ci/scripts/python_build\.sh$| ?^ci/scripts/python_sdist_build\.sh$| ?^ci/scripts/python_sdist_test\.sh$| + ?^ci/scripts/python_test_emscripten\.sh$| ?^ci/scripts/python_wheel_unix_test\.sh$| ?^ci/scripts/r_build\.sh$| ?^ci/scripts/r_revdepcheck\.sh$| diff --git a/ci/scripts/python_test_emscripten.sh b/ci/scripts/python_test_emscripten.sh index 4029722568b..621ac4bd0d1 100755 --- a/ci/scripts/python_test_emscripten.sh +++ b/ci/scripts/python_test_emscripten.sh @@ -25,14 +25,15 @@ set -ex build_dir=${1}/python pyodide_dist_dir=${2} -cd ${build_dir} +cd "${build_dir}" # note: this uses the newest wheel in dist +# shellcheck disable=SC2012 pyodide_wheel=$(ls -t dist/pyarrow*.whl | head -1) echo "-------------- Running emscripten tests in Node ----------------------" -python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=node +python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=node echo "-------------- Running emscripten tests in Chrome --------------------" -python scripts/run_emscripten_tests.py ${pyodide_wheel} --dist-dir=${pyodide_dist_dir} --runtime=chrome +python scripts/run_emscripten_tests.py "${pyodide_wheel}" --dist-dir="${pyodide_dist_dir}" --runtime=chrome