summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLeah Rowe <leah@libreboot.org>2023-09-01 08:46:24 +0100
committerLeah Rowe <leah@libreboot.org>2023-09-01 08:46:24 +0100
commit98f30b6dbe7ef38af543d571c333b54ad2709ad4 (patch)
tree591e392edea6999f191149b2571079d24206e357
parentb9662fbe3634af2946f0423a021c4f04c983c169 (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-xscript/build/coreboot/utils6
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" ] && \