diff options
author | Leah Rowe <leah@libreboot.org> | 2024-05-05 05:18:12 +0100 |
---|---|---|
committer | Leah Rowe <leah@libreboot.org> | 2024-05-05 05:18:12 +0100 |
commit | 580a5559480153f4b3e1983ccdce8e5a2c82c68d (patch) | |
tree | f8e021eb3df8abcc5d9eff17e976c5f3aa878628 | |
parent | 2fcbff68fc79112a06a76933c1d5fdb167c259b3 (diff) |
build/roms: don't use exit status from skip_board
the printf could potentially return non-zero, which might
make the script not skip a given target
Signed-off-by: Leah Rowe <leah@libreboot.org>
-rwxr-xr-x | script/build/roms | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/script/build/roms b/script/build/roms index 3d5a3f5c..81ecd879 100755 --- a/script/build/roms +++ b/script/build/roms @@ -95,9 +95,11 @@ handle_target() print_target_name && return 0 # exclude certain targets from the release - skip_board && \ - printf "Skip target %s(%s)\n" "$board" "$status" && \ - skipped="* $board($status)\n$skipped" && return 0 + if skip_board; then + printf "Skip target %s(%s)\n" "$board" "$status" + skipped="* $board($status)\n$skipped" + return 0 + fi build_payloads build_target_mainboard |