From 41bbf3673f579335257b7a2b1d369c668cc7e856 Mon Sep 17 00:00:00 2001 From: TryExceptElse Date: Fri, 3 Apr 2026 00:01:55 -0700 Subject: [PATCH] Replace 'ibs=' option with 'bs=' for better portability. I recently had to debug a failing makeself install which after some time and effort was traced to a minimal busybox `dd` implementation that lacked the 'ibs=' option. Since the only place where 'ibs=' is required is in the MS_dd_Progress() function, and the output is piped to /dev/null, my fix was to simply replace the 'ibs=' option with 'bs='. As the 'bs=' option is already being used several lines below this first call which jumps to the tar data, this places no new requirements upon systems. --- makeself-header.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/makeself-header.sh b/makeself-header.sh index 0b50d91..745256c 100755 --- a/makeself-header.sh +++ b/makeself-header.sh @@ -137,7 +137,7 @@ MS_dd_Progress() blocks=\`expr \$length / \$bsize\` bytes=\`expr \$length % \$bsize\` ( - dd ibs=\$offset skip=1 count=1 2>/dev/null + dd bs=\$offset skip=1 count=1 2>/dev/null pos=\`expr \$pos \+ \$bsize\` MS_Printf " 0%% " 1>&2 if test \$blocks -gt 0; then