diff options
| author | Leah Rowe <leah@libreboot.org> | 2025-05-19 19:57:06 +0100 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2025-05-19 19:58:55 +0100 |
| commit | 900da04efa929a9373d5a6f3f56f8b8ac8f193df (patch) | |
| tree | 893f758ce2917b7dea4e77311f4f292c9e50263a /include/tree.sh | |
| parent | 8aaf404ddea8a8d6c21f0a1f83f1675a8b0895aa (diff) | |
tree.sh: fix up copy_elf(), bad for loop
Because of how sh works, having just the [] line causes
sh to exit, annoyingly without an error message, but it
does cause a non-zero exit.
This bug will have already been triggering, before I added
the recent error handling on files for this for loop.
also do it to the other loop in lib.sh
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'include/tree.sh')
| -rw-r--r-- | include/tree.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/tree.sh b/include/tree.sh index 9dccf857..8d787d4c 100644 --- a/include/tree.sh +++ b/include/tree.sh @@ -334,7 +334,7 @@ copy_elf() { [ -f "$listfile" ] && x_ mkdir -p "$dest_dir" [ ! -f "$listfile" ] || while read -r f; do - [ -f "$srcdir/$f" ] && x_ cp "$srcdir/$f" "$dest_dir" - done < "$listfile" || err "copy_elf $*: cannot read '$listfile'" + [ -f "$srcdir/$f" ] && x_ cp "$srcdir/$f" "$dest_dir"; : + done < "$listfile" || err "copy_elf $*: cannot read '$listfile'"; : x_ make clean -C "$srcdir" $cleanargs } |