diff options
| author | Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org> | 2021-08-11 18:47:33 +0000 |
|---|---|---|
| committer | Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org> | 2021-08-11 18:50:14 +0000 |
| commit | 38e3fdfa8d60cb4455ba64503e193098e957c9dc (patch) | |
| tree | 1067eff530b2421b71b1041da15fbd6c39c54feb | |
| parent | 50d89f82e7795a151c4fae57f334f12b4d20f141 (diff) | |
lib/make.sh (make_check): Workaround GDB testsuite hanging abe.sh's teetcwg_gnu_native
See inline comment for details.
Change-Id: I7ee2f1be7b874952c88bd4164598f668479f4eb2
| -rw-r--r-- | lib/make.sh | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/lib/make.sh b/lib/make.sh index b6491c04..16802600 100644 --- a/lib/make.sh +++ b/lib/make.sh @@ -814,10 +814,18 @@ make_check() rm ${checklog} fi + notice "Redirecting output from the testsuite to $checklog" + local i result=0 for i in ${dirs}; do - # Always append "tee -a" to the log when building components individually - dryrun "make ${check_targets} FLAGS_UNDER_TEST=\"$test_flags\" PREFIX_UNDER_TEST=\"$prefix/bin/${target}-\" QEMU_CPU_UNDER_TEST=${qemu_cpu} ${schroot_make_opts} ${make_flags} -w -i -k -C ${builddir}$i 2>&1 | tee -a ${checklog}" + # Testsuites (I'm looking at you, GDB), can leave stray processes + # that inherit stdout of below "make check". Therefore, if we pipe + # stdout to "tee", then "tee" will wait on output from these + # processes for forever and ever. We workaround this by redirecting + # output to a file that can be "tail -f"'ed, if desired. + # A proper fix would be to fix dejagnu to not pass parent stdout + # to testcase processes. + dryrun "make ${check_targets} FLAGS_UNDER_TEST=\"$test_flags\" PREFIX_UNDER_TEST=\"$prefix/bin/${target}-\" QEMU_CPU_UNDER_TEST=${qemu_cpu} ${schroot_make_opts} ${make_flags} -w -i -k -C ${builddir}$i >> $checklog 2>&1" if [ $? != 0 ]; then warning "make ${check_targets} -C ${builddir}$i failed." result=1 |