From b298960aed8155e7dbedb6602cdbb42eacee83f7 Mon Sep 17 00:00:00 2001 From: Tianon Gravi Date: Mon, 7 Apr 2014 22:14:19 -0600 Subject: [PATCH] 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 (github: tianon) --- hack/make/binary | 1 + hack/make/cover | 1 + hack/make/cross | 1 + hack/make/dynbinary | 1 + hack/make/dyntest | 3 +-- hack/make/dyntest-integration | 3 +-- hack/make/test | 3 +-- hack/make/test-integration | 3 +-- hack/make/test-integration-cli | 3 +-- 9 files changed, 9 insertions(+), 10 deletions(-) diff --git a/hack/make/binary b/hack/make/binary index 041e4d1ee8..b97069a856 100755 --- a/hack/make/binary +++ b/hack/make/binary @@ -1,4 +1,5 @@ #!/bin/bash +set -e DEST=$1 diff --git a/hack/make/cover b/hack/make/cover index 6dc71d1c7e..ca772d03bc 100644 --- a/hack/make/cover +++ b/hack/make/cover @@ -1,4 +1,5 @@ #!/bin/bash +set -e DEST="$1" diff --git a/hack/make/cross b/hack/make/cross index e8f90e29b7..32fbbc38f9 100644 --- a/hack/make/cross +++ b/hack/make/cross @@ -1,4 +1,5 @@ #!/bin/bash +set -e DEST=$1 diff --git a/hack/make/dynbinary b/hack/make/dynbinary index 75cffe3dcc..426b9cb566 100644 --- a/hack/make/dynbinary +++ b/hack/make/dynbinary @@ -1,4 +1,5 @@ #!/bin/bash +set -e DEST=$1 diff --git a/hack/make/dyntest b/hack/make/dyntest index 744db3e999..56f624b1f5 100644 --- a/hack/make/dyntest +++ b/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 diff --git a/hack/make/dyntest-integration b/hack/make/dyntest-integration index ef7e6a5a41..03d7cbef95 100644 --- a/hack/make/dyntest-integration +++ b/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 diff --git a/hack/make/test b/hack/make/test index 39ba5cd3a5..183ce95c24 100644 --- a/hack/make/test +++ b/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 diff --git a/hack/make/test-integration b/hack/make/test-integration index 0af4c23c48..4c2bccaead 100644 --- a/hack/make/test-integration +++ b/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)" diff --git a/hack/make/test-integration-cli b/hack/make/test-integration-cli index b0506d261a..6e8b38892e 100644 --- a/hack/make/test-integration-cli +++ b/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}