What the developer wrote
Authored by Rusty Russell
tools: fix indentation.
Whitespace changes only.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
What changed, and why it matters
This commit only adjusts whitespace (tabs to spaces) in two build helper scripts. There is no change to program logic, no new behavior, and no security relevance.
No action required; this is a cosmetic/style commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff in tools/build-release.sh and tools/repro-build.sh is purely indentation normalization. The only non-whitespace-looking hunk in build-release.sh also only adds an environment variable (-e MAKEPAR) that was already present elsewhere in the script context; no functional or security change is introduced. No bug fixes, input handling changes, or executable logic modifications are present.
Changed components
tools/build-release.shtools/repro-build.shInspect captured patch +16 / −16
diff --git a/tools/build-release.sh b/tools/build-release.sh
index af215516..1f8c9846 100755
--- a/tools/build-release.sh
+++ b/tools/build-release.sh
@@ -172,13 +172,13 @@ for target in $TARGETS; do
Ubuntu*)
distributions=${platform#Ubuntu-}
[ "$distributions" = "Ubuntu" ] && distributions="focal jammy noble"
- for d in $distributions; do
+ for d in $distributions; do
# Capitalize the first letter of distro
D=$(echo "$d" | awk '{print toupper(substr($0,1,1))substr($0,2)}')
- echo "Building Ubuntu $D Image"
- docker run --rm -v "$(pwd)":/repo -e FORCE_MTIME="$MTIME" -e FORCE_VERSION="$VERSION" cl-repro-"$d"
+ echo "Building Ubuntu $D Image"
+ docker run --rm -v "$(pwd)":/repo -e FORCE_MTIME="$MTIME" -e FORCE_VERSION="$VERSION" -e MAKEPAR="$MAKEPAR" cl-repro-"$d"
echo "Ubuntu $D Image Built"
- done
+ done
;;
*)
echo "No Dockerfile for $platform" >&2
diff --git a/tools/repro-build.sh b/tools/repro-build.sh
index b5a3af61..0d0963da 100755
--- a/tools/repro-build.sh
+++ b/tools/repro-build.sh
@@ -9,19 +9,19 @@ export LANG LC_ALL
for arg; do
case "$arg" in
--force-version=*)
- FORCE_VERSION=${arg#*=}
+ FORCE_VERSION=${arg#*=}
;;
- --force-mtime=*)
- FORCE_MTIME=${arg#*=}
- ;;
- --help)
- echo "Usage: [--force-version=<ver>] [--force-mtime=YYYY-MM-DD]"
- exit 0
- ;;
- *)
- echo "Unknown arg $arg" >&2
- exit 1
- ;;
+ --force-mtime=*)
+ FORCE_MTIME=${arg#*=}
+ ;;
+ --help)
+ echo "Usage: [--force-version=<ver>] [--force-mtime=YYYY-MM-DD]"
+ exit 0
+ ;;
+ *)
+ echo "Unknown arg $arg" >&2
+ exit 1
+ ;;
esac
shift
done
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.