From 652c7aa2b17920f91803efcc2f18bf1991a1c091 Mon Sep 17 00:00:00 2001
From: Jess Frazelle <me@jessfraz.com>
Date: Mon, 4 Jul 2016 09:46:27 -0700
Subject: [PATCH] update build all

Signed-off-by: Jess Frazelle <me@jessfraz.com>
---
 build-all.sh | 68 ++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 48 insertions(+), 20 deletions(-)

diff --git a/build-all.sh b/build-all.sh
index 0da4364..e5efd47 100755
--- a/build-all.sh
+++ b/build-all.sh
@@ -1,25 +1,10 @@
 #!/bin/bash
 set -e
 
-# get the dockerfiles
-IFS=$'\n'
-files=( $(find . -iname '*Dockerfile' | sed 's|./||') )
-unset IFS
-
-# build the changed dockerfiles
-for f in "${files[@]}"; do
-	image=${f%Dockerfile}
-	base=${image%%\/*}
-	build_dir=$(dirname $f)
-	suite=${build_dir##*\/}
-
-	if [[ -z "$suite" ]] || [[ "$suite" == "$base" ]]; then
-		suite=latest
-	fi
-
-	if [[ "${base}" == "sup" ]]; then
-		continue
-	fi
+build_and_push(){
+	base=$1
+	suite=$2
+	build_dir=$3
 
 	(
 	set -x
@@ -45,4 +30,47 @@ for f in "${files[@]}"; do
 		docker tag r.j3ss.co/${base}:${suite} r.j3ss.co/${base}:latest
 		docker push --disable-content-trust=false r.j3ss.co/${base}:latest
 	fi
-done
+
+}
+
+main(){
+	# get the dockerfiles
+	IFS=$'\n'
+	files=( $(find . -iname '*Dockerfile' | sed 's|./||') )
+	unset IFS
+
+	ERRORS=()
+	# build all dockerfiles
+	for f in "${files[@]}"; do
+		image=${f%Dockerfile}
+		base=${image%%\/*}
+		build_dir=$(dirname $f)
+		suite=${build_dir##*\/}
+
+		if [[ -z "$suite" ]] || [[ "$suite" == "$base" ]]; then
+			suite=latest
+		fi
+
+		if [[ "${base}" == "sup" ]]; then
+			continue
+		fi
+
+		{
+			build_and_push "${base}" "${suite}" "${build_dir}"
+		} || {
+			# add to errors
+			ERRORS+=("${base}:${suite}")
+		}
+		echo
+		echo
+	done
+
+	if [ ${#ERRORS[@]} -eq 0 ]; then
+    	echo "No errors, hooray"
+	else
+    	echo "These images failed: ${ERRORS[@]}"
+		exit 1
+	fi
+}
+
+main $@