Browse Source

Merge pull request #5890 from LK4D4/fix_tee_hanging_on_test_fails_#5672

Fix hanging on tests failing
Tianon Gravi 11 years ago
parent
commit
05c3325a02
3 changed files with 6 additions and 4 deletions
  1. 2 2
      hack/make/test-integration
  2. 2 1
      hack/make/test-integration-cli
  3. 2 1
      hack/make/test-unit

+ 2 - 2
hack/make/test-integration

@@ -10,6 +10,6 @@ bundle_test_integration() {
 
 
 # this "grep" hides some really irritating warnings that "go test -coverpkg"
 # this "grep" hides some really irritating warnings that "go test -coverpkg"
 # spews when it is given packages that aren't used
 # spews when it is given packages that aren't used
+exec > >(tee -a $DEST/test.log) 2>&1
 bundle_test_integration 2>&1 \
 bundle_test_integration 2>&1 \
-	| grep --line-buffered -v '^warning: no packages being tested depend on ' \
-	| tee $DEST/test.log
+	| grep --line-buffered -v '^warning: no packages being tested depend on '

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

@@ -11,6 +11,7 @@ bundle_test_integration_cli() {
 }
 }
 
 
 # subshell so that we can export PATH without breaking other things
 # subshell so that we can export PATH without breaking other things
+exec > >(tee -a $DEST/test.log) 2>&1
 (
 (
 	export PATH="$DEST/../binary:$DEST/../dynbinary:$PATH"
 	export PATH="$DEST/../binary:$DEST/../dynbinary:$PATH"
 
 
@@ -40,4 +41,4 @@ bundle_test_integration_cli() {
 	DOCKERD_PID=$(set -x; cat $DEST/docker.pid)
 	DOCKERD_PID=$(set -x; cat $DEST/docker.pid)
 	( set -x; kill $DOCKERD_PID )
 	( set -x; kill $DOCKERD_PID )
 	wait $DOCKERD_PID || true
 	wait $DOCKERD_PID || true
-) 2>&1 | tee $DEST/test.log
+)

+ 2 - 1
hack/make/test-unit

@@ -49,7 +49,8 @@ bundle_test_unit() {
 			echo
 			echo
 			true
 			true
 		fi
 		fi
-	} 2>&1 | tee $DEST/test.log
+	}
 }
 }
 
 
+exec > >(tee -a $DEST/test.log) 2>&1
 bundle_test_unit
 bundle_test_unit