From 5abde443b2bdae7723b046f3db3b8e821ab67cde Mon Sep 17 00:00:00 2001
From: Andrew Robbins <contact@andrewrobbins.info>
Date: Sat, 22 Jul 2017 02:10:25 -0400
Subject: Make 'sources' consequent prettier

Removes an unnecessary subshell grouping and 'set' statement and
replaces it with a good ol' if/then/else construct.  Also added a
message printed to stdout informing the user of the change from
extraction to downloading if source extraction fails.
---
 libreboot | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'libreboot')

diff --git a/libreboot b/libreboot
index 59795d09..73176858 100755
--- a/libreboot
+++ b/libreboot
@@ -78,12 +78,12 @@ libreboot_project() {
 
 	case "${action}" in
 		'sources')
-			(
-				set +e
-
-				project_action_arguments 'extract' "${project}" "$@" && return 0
-				project_action_arguments 'download' "${project}" "$@" && return 0
-			)
+			if project_action_arguments 'extract' "${project}" "$@"; then
+				return
+			else
+				printf '\n%s\n\n' 'Attempting to download instead...'
+				project_action_arguments 'download' "${project}" "$@"
+			fi
 			;;
 		'produce')
 			for action in 'build' 'install' 'release'; do
-- 
cgit v1.2.3-70-g09d2