Commit 3676834a by Edward Thomson Committed by Patrick Steinhardt

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.

(cherry picked from commit 6eb97b6b)
parent 8a54e39c
...@@ -12,7 +12,7 @@ TMPDIR=${TMPDIR:-/tmp} ...@@ -12,7 +12,7 @@ TMPDIR=${TMPDIR:-/tmp}
USER=${USER:-$(whoami)} USER=${USER:-$(whoami)}
VALGRIND="valgrind --leak-check=full --show-reachable=yes --error-exitcode=125 --num-callers=50 --suppressions=\"$SOURCE_DIR/libgit2_clar.supp\"" 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() { cleanup() {
echo "Cleaning up..." echo "Cleaning up..."
...@@ -47,7 +47,6 @@ run_test() { ...@@ -47,7 +47,6 @@ run_test() {
RUNNER="$VALGRIND $TEST_CMD" RUNNER="$VALGRIND $TEST_CMD"
elif [ "$LEAK_CHECK" = "leaks" ]; then elif [ "$LEAK_CHECK" = "leaks" ]; then
RUNNER="$LEAKS $TEST_CMD" RUNNER="$LEAKS $TEST_CMD"
echo $RUNNER
else else
RUNNER="$TEST_CMD" RUNNER="$TEST_CMD"
fi fi
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment