From 98f30b6dbe7ef38af543d571c333b54ad2709ad4 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Fri, 1 Sep 2023 08:46:24 +0100 Subject: 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 --- script/build/coreboot/utils | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'script') 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" ] && \ -- cgit v1.2.1