Browse Source

Update bundlescripts to use "set -e" consistently

"set -e" is already inherited here from make.sh, but explicit is always better than implicit (hence the "set -e" in the first place!)

Docker-DCO-1.1-Signed-off-by: Andrew Page <admwiggin@gmail.com> (github: tianon)
Tianon Gravi 11 years ago
parent
commit
b298960aed

+ 1 - 0
hack/make/binary

@@ -1,4 +1,5 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 

+ 1 - 0
hack/make/cover

@@ -1,4 +1,5 @@
 #!/bin/bash
+set -e
 
 DEST="$1"
 

+ 1 - 0
hack/make/cross

@@ -1,4 +1,5 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 

+ 1 - 0
hack/make/dynbinary

@@ -1,4 +1,5 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 

+ 1 - 2
hack/make/dyntest

@@ -1,10 +1,9 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 INIT=$DEST/../dynbinary/dockerinit-$VERSION
 
-set -e
-
 if [ ! -x "$INIT" ]; then
 	echo >&2 'error: dynbinary must be run before dyntest'
 	false

+ 1 - 2
hack/make/dyntest-integration

@@ -1,10 +1,9 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 INIT=$DEST/../dynbinary/dockerinit-$VERSION
 
-set -e
-
 if [ ! -x "$INIT" ]; then
 	echo >&2 'error: dynbinary must be run before dyntest-integration'
 	false

+ 1 - 2
hack/make/test

@@ -1,9 +1,8 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 
-set -e
-
 RED=$'\033[31m'
 GREEN=$'\033[32m'
 TEXTRESET=$'\033[0m' # reset the foreground colour

+ 1 - 2
hack/make/test-integration

@@ -1,9 +1,8 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 
-set -e
-
 bundle_test_integration() {
 	LDFLAGS="$LDFLAGS $LDFLAGS_STATIC_DOCKER" go_test_dir ./integration \
 		"-coverpkg $(find_dirs '*.go' | sed 's,^\.,github.com/dotcloud/docker,g' | paste -d, -s)"

+ 1 - 2
hack/make/test-integration-cli

@@ -1,9 +1,8 @@
 #!/bin/bash
+set -e
 
 DEST=$1
 
-set -e
-
 DOCKER_GRAPHDRIVER=${DOCKER_GRAPHDRIVER:-vfs}
 DOCKER_EXECDRIVER=${DOCKER_EXECDRIVER:-native}