From 86512e84bea014a9b9e71b1a5adee93810cba411 Mon Sep 17 00:00:00 2001
From: Leah Rowe <leah@libreboot.org>
Date: Mon, 15 May 2023 02:38:22 +0100
Subject: [PATCH] download/coreboot: simplify small if statements

---
 resources/scripts/download/coreboot | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/resources/scripts/download/coreboot b/resources/scripts/download/coreboot
index e0c55c9..d8b44ab 100755
--- a/resources/scripts/download/coreboot
+++ b/resources/scripts/download/coreboot
@@ -84,26 +84,21 @@ download_coreboot_for_board()
 	if [ -d "coreboot/${cbtree}" ]; then
 		printf "REMARK: download/coreboot %s: exists. Skipping.\n" \
 				${cbtree}
-		if [ "${cbtree}" != "${1}" ]; then
-			printf "(for board: '${1}')\n"
-		fi
+		[ "${cbtree}" != "${1}" ] && \
+			printf "(for board: '%s}')\n" ${1}
 		return 0
 	fi
 
-	if [ ! -d coreboot ]; then
+	[ ! -d coreboot ] && \
 		mkdir "coreboot/"
-	fi
-	if [ ! -d coreboot ]; then
-		printf "ERROR: download/coreboot: directory not created."
-		printf " Check file system permissions\n"
+	[ ! -d coreboot ] && \
+		printf "ERROR: download/coreboot: directory not created\n" && \
 		return 1
-	fi
 
 	cd "coreboot/"
 
-	if [ ! -d coreboot/.git ] && [ -d coreboot ]; then
+	[ ! -d coreboot/.git ] && [ -d coreboot ] && \
 		rm -Rf coreboot/
-	fi
 
 	if [ ! -d coreboot ]; then
 		printf "Download coreboot from upstream:\n"
-- 
GitLab