diff options
| author | Leah Rowe <leah@libreboot.org> | 2023-09-01 08:46:24 +0100 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2023-09-01 08:46:24 +0100 |
| commit | 98f30b6dbe7ef38af543d571c333b54ad2709ad4 (patch) | |
| tree | 591e392edea6999f191149b2571079d24206e357 | |
| parent | b9662fbe3634af2946f0423a021c4f04c983c169 (diff) | |
build/coreboot/utils: exit 1 if target.cfg missing
it was previously trying to "continue", despite not being
inside a loop. the correct instruction would have
been "return 0", but then I thought it'd be better to
err here
Signed-off-by: Leah Rowe <leah@libreboot.org>
| -rwxr-xr-x | script/build/coreboot/utils | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/script/build/coreboot/utils b/script/build/coreboot/utils index f7af680b..80711fae 100755 --- a/script/build/coreboot/utils +++ b/script/build/coreboot/utils @@ -43,8 +43,10 @@ main() build_for_mainboard() { board="${1}" - [ -d "resources/coreboot/${board}" ] || continue - [ -f "resources/coreboot/${board}/target.cfg" ] || continue + [ -d "resources/coreboot/${board}" ] || \ + err "build_for_mainboard ${board}: boarddir does not exist" + [ -f "resources/coreboot/${board}/target.cfg" ] || \ + err "build_for_mainboard ${board}: target.cfg does not exist" tree="undefined" . "resources/coreboot/${board}/target.cfg" # source [ "${tree}" = "undefined" ] && \ |