On 6/12/19 10:52 PM, Eduardo Habkost wrote: > On Wed, Jun 12, 2019 at 10:49:18PM +0200, Philippe Mathieu-Daudé wrote: >> On 6/11/19 7:14 PM, Eduardo Habkost wrote: >>> It will help us debug issues when tests fail. >>> >>> Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> >>> --- >>> .travis.yml | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/.travis.yml b/.travis.yml >>> index b053a836a3..a08a7b7278 100644 >>> --- a/.travis.yml >>> +++ b/.travis.yml >>> @@ -225,7 +225,7 @@ matrix: >>> # Acceptance (Functional) tests >>> - env: >>> - CONFIG="--python=/usr/bin/python3 >>> --target-list=x86_64-softmmu,mips-softmmu,mips64el-softmmu,aarch64-softmmu,arm-softmmu,s390x-softmmu,alpha-softmmu" >>> - - TEST_CMD="make check-acceptance" >>> + - TEST_CMD="make AVOCADO_SHOW=test check-acceptance" >>> addons: >>> apt: >>> packages: >>> >> >> Cleber, can you help figuring what's wrong here? >> >> https://travis-ci.org/qemu/qemu/jobs/544805900 >> >> The tests seems successfull, is it possible the verbose logging returns >> error to the shell even if the test succeed? > > That's exactly what this patch does. On my next pull request I > will replace this with Cleber's patch that dumps the error log > only if tests failed.
Ah OK, this is annoying because now all Travis builds are marked as failed. Maybe Peter can apply your patch directly to master as a build fix?
