diff options
| author | Leah Rowe <leah@libreboot.org> | 2023-11-08 06:31:04 +0000 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2023-11-08 06:34:12 +0000 |
| commit | 5af3ae0586ba1f437cecab551ab4fba6fb6bdef1 (patch) | |
| tree | 3c7e53042fb78f87da8d7a886ec558993baa54ef /script/build/serprog | |
| parent | 64f933747021cb783230b9aef367921884ece555 (diff) | |
lbmk: don't use status for unconditional returns
in cases where lbmk must always return from a function,
there are some cases where it relies on non-zero exit
status, which in practise is always the case, but may
change in the future if the relevant part is modified
e.g. do_something && return 0
the proper form is:
do_something
return 0
also do this for unconditional exits
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'script/build/serprog')
| -rwxr-xr-x | script/build/serprog | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/script/build/serprog b/script/build/serprog index 93ea9302..7f8664a9 100755 --- a/script/build/serprog +++ b/script/build/serprog @@ -34,7 +34,10 @@ main() x_ mkdir -p "bin/serprog_${platform}" if [ $# -gt 1 ] ; then - [ "${2}" = "list" ] && print_boards ${boards_dir} && return 0 + if [ "${2}" = "list" ]; then + print_boards ${boards_dir} + return 0 + fi build_${platform}_rom "${2}" else printf "Building all serprog targets\n" |