ci: Drop special handling of test-suite.log
authorDan Nicholson <dbn@endlessos.org>
Mon, 21 Jun 2021 19:42:20 +0000 (13:42 -0600)
committerDan Nicholson <dbn@endlessos.org>
Mon, 21 Jun 2021 19:47:00 +0000 (13:47 -0600)
The automake test harness[1] already dumps the contents of
`test-suite.log` when `VERBOSE=1` is set, so we don't need to add
special handling for it.

1. https://www.gnu.org/software/automake/manual/html_node/Parallel-Test-Harness.html

ci/gh-build.sh

index 949ce5facd3d7a1a1d1ee33d79f69e19bf8c3d36..11434327e5b8ce291fd16afe1dc6b3b31f52a19f 100755 (executable)
@@ -32,6 +32,8 @@ srcdir="$(pwd)"
 mkdir ci-build
 cd ci-build
 
+# V=1 shows the full build commands. VERBOSE=1 dumps test-suite.log on
+# failures.
 make="make V=1 VERBOSE=1"
 
 ../configure \
@@ -40,20 +42,9 @@ make="make V=1 VERBOSE=1"
 
 ${make}
 
-# Run the tests both using check and distcheck and dump the logs on
-# failures. For distcheck the logs will be inside the dist directory, so
-# tell make to use the current directory.
-if ! ${make} check; then
-    cat test-suite.log || :
-    exit 1
-fi
-if ! ${make} distcheck \
-    TEST_SUITE_LOG=$(pwd)/test-suite.log \
-    DISTCHECK_CONFIGURE_FLAGS="$*"
-then
-    cat test-suite.log || :
-    exit 1
-fi
+# Run the tests both using check and distcheck.
+${make} check
+${make} distcheck DISTCHECK_CONFIGURE_FLAGS="$*"
 
 # Show the installed files
 ${make} install DESTDIR=$(pwd)/DESTDIR