From 2b4bc5eff0c3b7cbb1d5b50c62eaf02f3f9badab Mon Sep 17 00:00:00 2001
From: Luke Shumaker <lukeshu@lukeshu.com>
Date: Wed, 24 May 2017 00:03:39 -0400
Subject: librerelease: Be more careful with config parsing (fixes test)

---
 src/abslibre-tools/librerelease | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/abslibre-tools/librerelease b/src/abslibre-tools/librerelease
index b8eb0e3..41eb21b 100755
--- a/src/abslibre-tools/librerelease
+++ b/src/abslibre-tools/librerelease
@@ -165,13 +165,23 @@ main() {
 		return 0
 	fi
 
-	load_files makepkg
-	check_vars makepkg GPGKEY
-	load_files libretools
-	check_vars libretools WORKDIR REPODEST ABSLIBREDEST || return 1
+	# defer exiting on errors until we've tried to parse *all*
+	# config files, to have more complete error messages.
+	local err=false
+	{
+		load_files makepkg &&
+		check_vars makepkg GPGKEY
+	} || err=true
+	{
+		load_files libretools &&
+		# the HOOK* are settings are optional
+		check_vars libretools WORKDIR REPODEST ABSLIBREDEST # HOOKPRERELEASE HOOKPOSTRELEASE
+	} || err=true
+	if $err; then
+		return 6
+	fi
+
 	REPODEST+='/staging/'
-	# The following settings are actually optional
-	#check_vars libretools HOOKPRERELEASE HOOKPOSTRELEASE || return 1
 
 	"$mode"
 }
-- 
cgit v1.2.3-54-g00ecf