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.

1 files changed, 6 insertions, 6 deletions

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