[ARVADOS] updated: e47bb9c1d07dc67d0a3609a494a91c32ecbe6d85

[Git user]

Summary of changes:
 build/run-tests.sh | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

       via  e47bb9c1d07dc67d0a3609a494a91c32ecbe6d85 (commit)
      from  17f4a03d71d8d6130f796d61fa49b8480bf555b6 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e47bb9c1d07dc67d0a3609a494a91c32ecbe6d85
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Feb 24 17:46:38 2017 -0500

    run-tests.sh exit non-zero if gofmt fails
    
    No issue #

diff --git a/build/run-tests.sh b/build/run-tests.sh
index d922985..44f9a30 100755
--- a/build/run-tests.sh
+++ b/build/run-tests.sh
@@ -548,14 +548,14 @@ do_test_once() {
         # before trying "go test". Otherwise, coverage-reporting
         # mode makes Go show the wrong line numbers when reporting
         # compilation errors.
-        go get -t "git.curoverse.com/arvados.git/$1" || return 1
-        cd "$WORKSPACE/$1" || return 1
-        gofmt -e -d . | egrep . && result=1
-        if [[ -n "${testargs[$1]}" ]]
+        go get -t "git.curoverse.com/arvados.git/$1" && \
+            cd "$WORKSPACE/$1" && \
+            [[ -z "$(gofmt -e -d . | tee /dev/stderr)" ]] && \
+            if [[ -n "${testargs[$1]}" ]]
         then
             # "go test -check.vv giturl" doesn't work, but this
             # does:
-            cd "$WORKSPACE/$1" && go test ${short:+-short} ${testargs[$1]}
+            go test ${short:+-short} ${testargs[$1]}
         else
             # The above form gets verbose even when testargs is
             # empty, so use this form in such cases:

-----------------------------------------------------------------------


hooks/post-receive
--