Skip to content

Commit 416e3f1

Browse files
authored
chore: address commit comments for commit be0ff01
PR-URL: #11008 Closes: #11005 Reviewed-by: Philipp Burckhardt <pburckhardt@outlook.com>
1 parent a410628 commit 416e3f1

File tree

2 files changed

+4
-5
lines changed
  • .github/workflows/scripts
    • run_affected_benchmarks
    • run_affected_tests

2 files changed

+4
-5
lines changed

.github/workflows/scripts/run_affected_benchmarks/run

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ main() {
142142
# Run JS benchmarks:
143143
if [ -n "${js_bench_files}" ]; then
144144
# Invoke make in batches to avoid "Argument list too long" errors:
145-
printf '%s\n' ${js_bench_files} | xargs -s 65536 sh -c 'make benchmark-javascript-files FILES="$*"' _
145+
printf '%s\n' ${js_bench_files} | xargs sh -c 'make benchmark-javascript-files FILES="$*"' _
146146
else
147147
echo 'No JavaScript benchmarks to run.' >&2
148148
fi
@@ -174,7 +174,7 @@ main() {
174174

175175
if [ -n "${c_bench_files}" ]; then
176176
# Invoke make in batches to avoid "Argument list too long" errors:
177-
printf '%s\n' ${c_bench_files} | xargs -s 65536 sh -c 'make benchmark-c-files FILES="$*"' _
177+
printf '%s\n' ${c_bench_files} | xargs sh -c 'make benchmark-c-files FILES="$*"' _
178178
else
179179
echo 'No C benchmarks to run.' >&2
180180
fi

.github/workflows/scripts/run_affected_tests/run

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,9 +165,8 @@ main() {
165165
files=$(echo "${files}" | grep -v '/fixtures/') || true
166166

167167
if [[ -n "${files}" ]]; then
168-
# Invoke make in batches to avoid "Argument list too long" errors
169-
# when the total length of file paths exceeds OS limits:
170-
printf '%s\n' ${files} | xargs -s 65536 sh -c 'make test-javascript-files-min FILES="$*"' _
168+
# Invoke make in batches to avoid "Argument list too long" errors:
169+
printf '%s\n' ${files} | xargs sh -c 'make test-javascript-files-min FILES="$*"' _
171170
fi
172171

173172
cleanup

0 commit comments

Comments
 (0)