Skip to content

Commit b072a5f

Browse files
committed
Do not run fuzzer in subprocess
There is no obvious reason to run the fuzzer in a subprocess. Remove the subprocess from the fuzzing code. While we are at it change the code comment to state what it does instead of mentioning the CI vms.
1 parent 62d5169 commit b072a5f

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

contrib/test.sh

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,12 @@ fi
2525
# Fuzz if told to
2626
if [ "$DO_FUZZ" = true ]
2727
then
28-
(
29-
cd fuzz
30-
cargo test --verbose
31-
./travis-fuzz.sh
32-
# Exit out of the fuzzer,
33-
# run stable tests in other CI vms
34-
exit 0
35-
)
28+
cd fuzz
29+
cargo test --verbose
30+
./travis-fuzz.sh
31+
32+
# Exit out of the fuzzer, do not run other tests.
33+
exit 0
3634
fi
3735

3836
# Test without any features first

0 commit comments

Comments
 (0)