Re: [Qemu-devel] [PULL 00/20] Travis, Code Coverage and Cross Build updates

[Paolo Bonzini]

On 03/07/2018 12:14, Alex Bennée wrote:
> The following changes since commit 46d0885adff9b99622d72f23a8b04c298a8bf91d:
> 
>   Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging 
> (2018-07-03 09:49:20 +0100)
> 
> are available in the Git repository at:
> 
>   https://github.com/stsquad/qemu.git 
> tags/pull-code-coverage-and-build-tweaks-030718-1
> 
> for you to fetch changes up to e84a4f16f0fd67e5e503ceb50a9d069f7b76fcb3:
> 
>   docker: add linux-user powered cross builder for QEMU (2018-07-03 11:09:56 
> +0100)
> 
> ----------------------------------------------------------------
> Code coverage and other build tweaks
> 
>   - some travis speed-ups
>   - modernise code coverage support
>   - docker image cleanups
>   - clean-up binfmt_misc docker infrastructure
>   - add debian-powerpc-user-cross image for ppc32 build
> 
> ----------------------------------------------------------------
> Alex Bennée (14):
>       build-system: remove per-test GCOV reporting
>       .gitignore: add .gcov files
>       docker: add gcovr to travis image
>       travis: add gcovr summary for GCOV build
>       build-system: add clean-coverage target
>       build-system: add coverage-report target
>       linux-user: introduce preexit_cleanup
>       linux-user: add gcov support to preexit_cleanup
>       docker: filter out linux-user builds for mingw
>       docker: drop QEMU build-dep from bootstrap
>       docker: debian-bootstrap.pre allow customising of variant/url
>       docker: add special handling for FROM:debian-%-user targets
>       docker: add special rule for deboostrapped images
>       docker: add linux-user powered cross builder for QEMU
> 
> Philippe Mathieu-Daudé (6):
>       travis: do not waste time cloning unused submodules
>       travis: test out-of-tree builds
>       docker: ubuntu: Update the package list before installing new ones
>       docker: ubuntu: Use SDL2
>       docker: Clean the MXE base image
>       docker: Do not run tests in 'intermediate' images
> 
>  .gitignore                                         |  1 +
>  .travis.yml                                        | 14 +++-
>  MAINTAINERS                                        |  1 +
>  Makefile                                           | 24 +++++++
>  docs/devel/testing.rst                             | 21 ++++--
>  linux-user/Makefile.objs                           |  2 +-
>  linux-user/exit.c                                  | 35 ++++++++++
>  linux-user/qemu.h                                  |  8 +++
>  linux-user/syscall.c                               | 10 +--
>  scripts/travis/coverage-summary.sh                 | 27 ++++++++
>  tests/Makefile.include                             | 10 ---
>  tests/docker/Makefile.include                      | 74 
> ++++++++++++++++++++--
>  tests/docker/docker.py                             |  4 ++
>  tests/docker/dockerfiles/debian-bootstrap.docker   |  2 -
>  tests/docker/dockerfiles/debian-bootstrap.pre      | 11 +++-
>  .../dockerfiles/debian-powerpc-user-cross.docker   | 15 +++++
>  tests/docker/dockerfiles/debian8-mxe.docker        |  2 +-
>  tests/docker/dockerfiles/travis.docker             |  2 +-
>  tests/docker/dockerfiles/ubuntu.docker             |  8 +--
>  19 files changed, 228 insertions(+), 43 deletions(-)
>  create mode 100644 linux-user/exit.c
>  create mode 100755 scripts/travis/coverage-summary.sh
>  create mode 100644 tests/docker/dockerfiles/debian-powerpc-user-cross.docker
> 
> --
> 2.17.1
> 
> 

A bit too fast perhaps...  Can we just revert the offending patch
instead of patch 11?

Thanks,

Paolo