Commit 6eb97b6ba93019741e7cf6147f0fab05dd3f831d

Edward Thomson 2018-07-25T01:04:55

ci: dissociate test from leaks process The leaks process is not good about handling children. Ensure that its child is `nohup`ed so that the grandparent shell won't wait for it to exit.

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
diff --git a/ci/test.sh b/ci/test.sh
index 013476a..28f7602 100755
--- a/ci/test.sh
+++ b/ci/test.sh
@@ -12,7 +12,7 @@ TMPDIR=${TMPDIR:-/tmp}
 USER=${USER:-$(whoami)}
 
 VALGRIND="valgrind --leak-check=full --show-reachable=yes --error-exitcode=125 --num-callers=50 --suppressions=\"$SOURCE_DIR/libgit2_clar.supp\""
-LEAKS="MallocStackLogging=1 MallocScribble=1 leaks -quiet -atExit --"
+LEAKS="MallocStackLogging=1 MallocScribble=1 leaks -quiet -atExit -- nohup"
 
 cleanup() {
 	echo "Cleaning up..."
@@ -47,7 +47,6 @@ run_test() {
 		RUNNER="$VALGRIND $TEST_CMD"
 	elif [ "$LEAK_CHECK" = "leaks" ]; then
 		RUNNER="$LEAKS $TEST_CMD"
-		echo $RUNNER
 	else
 		RUNNER="$TEST_CMD"
 	fi