From 40fe79938b5f5674c76c1c49967df749ab1de66e Mon Sep 17 00:00:00 2001 From: Ben Kunkle Date: Wed, 21 Jan 2026 10:10:18 -0600 Subject: [PATCH] Use arithmetic expansion instead of `expr` in `./script/clear-target-dir-if-larger-than` (#47298) Fixes CI failures when the target dir is 0GB in size, which causes `expr` to output a non-zero exit code per the posix spec. See exit status section of https://www.man7.org/linux/man-pages/man1/expr.1p.html Release Notes: - N/A *or* Added/Fixed/Improved ... --- script/clear-target-dir-if-larger-than | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/script/clear-target-dir-if-larger-than b/script/clear-target-dir-if-larger-than index 46256159a804aff14422dc7d205e65016bf5a83a..6041c0f5be00946a78b7659b1e08b8d57e2e4e36 100755 --- a/script/clear-target-dir-if-larger-than +++ b/script/clear-target-dir-if-larger-than @@ -15,7 +15,7 @@ fi max_size_gb=$1 current_size=$(du -s target | cut -f1) -current_size_gb=$(expr ${current_size} / 1024 / 1024) +current_size_gb=$(( ${current_size} / 1024 / 1024 )) echo "target directory size: ${current_size_gb}gb. max size: ${max_size_gb}gb"