[Qemu-commits] [qemu/qemu] 628384: docker: add installation to build tests

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 6283847857d8008046ceddf2db4bd908df0c063a
      
https://github.com/qemu/qemu/commit/6283847857d8008046ceddf2db4bd908df0c063a
  Author: Paolo Bonzini <address@hidden>
  Date:   2017-09-29 (Fri, 29 Sep 2017)

  Changed paths:
    M tests/docker/common.rc
    M tests/docker/dockerfiles/centos6.docker
    M tests/docker/dockerfiles/centos7.docker
    M tests/docker/dockerfiles/debian-ports.docker
    M tests/docker/dockerfiles/debian8.docker
    M tests/docker/dockerfiles/debian9.docker
    M tests/docker/dockerfiles/fedora.docker
    M tests/docker/dockerfiles/min-glib.docker
    M tests/docker/dockerfiles/ubuntu.docker
    M tests/docker/test-build
    M tests/docker/test-clang
    M tests/docker/test-full
    M tests/docker/test-mingw
    M tests/docker/test-quick

  Log Message:
  -----------
  docker: add installation to build tests

Basic test that "make install" works; this requires msgfmt so add
gettext to the packages.

Signed-off-by: Paolo Bonzini <address@hidden>
Message-Id: <address@hidden>
[Rebase to master. - Fam]
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 299d296ea90cca388b3a1fb316c797da5b6a6708
      
https://github.com/qemu/qemu/commit/299d296ea90cca388b3a1fb316c797da5b6a6708
  Author: Fam Zheng <address@hidden>
  Date:   2017-09-29 (Fri, 29 Sep 2017)

  Changed paths:
    M tests/docker/run

  Log Message:
  -----------
  docker: Fix test-mingw

Feature "dtc" is explicitly required by test-mingw, but is not detected
by the run script since we switched to archive-source.sh in b7f404201e4.
Since it isn't available in the Fedora image which runs this test on
patchew, the way we get dtc is still from submodule.

archive-source.sh takes care of bundling the submodule files already, so
what we need to do is just checking if files are there. Makefile is
chosen because it is one that is unlikely to get renamed in the future.

Signed-off-by: Fam Zheng <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: bcd7f06f57fb6f780a3e2f7a46c22b6f6c8238aa
      
https://github.com/qemu/qemu/commit/bcd7f06f57fb6f780a3e2f7a46c22b6f6c8238aa
  Author: Alex Bennée <address@hidden>
  Date:   2017-09-29 (Fri, 29 Sep 2017)

  Changed paths:
    M tests/docker/run

  Log Message:
  -----------
  tests/docker/run: don't source /etc/profile

The usual behaviour of /etc/profile is to set the default PATH for
users. This runs into problems when we have updated PATH in our
dockerfile e.g. to access a cross-compiler in a non-standard
location. It shouldn't be needed anyway as we inherit the env from the
image when it was setup.

Signed-off-by: Alex Bennée <address@hidden>
CC: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 12b25a7d96dc01fc9710c4af5942c15a51b84db1
      
https://github.com/qemu/qemu/commit/12b25a7d96dc01fc9710c4af5942c15a51b84db1
  Author: Fam Zheng <address@hidden>
  Date:   2017-09-29 (Fri, 29 Sep 2017)

  Changed paths:
    M tests/docker/test-block

  Log Message:
  -----------
  docker: test-block: Don't continue if build fails

Report error and exit upon compiling error, otherwise the iotests output
will be pure noise.

Signed-off-by: Fam Zheng <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 36ac78e65a0695f0e397865cceaf4ab5c4e52c64
      
https://github.com/qemu/qemu/commit/36ac78e65a0695f0e397865cceaf4ab5c4e52c64
  Author: Fam Zheng <address@hidden>
  Date:   2017-09-29 (Fri, 29 Sep 2017)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  docker: Don't mount ccache db if NOUSER=1

With NOUSER=1 the container runs code as root, which may create
privileged files that will not be be accssible next time. Skip ccache
dir mount in this case.

Signed-off-by: Fam Zheng <address@hidden>
Message-Id: <address@hidden>
Acked-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>


  Commit: be9d199751789fdc96a3febe3f0768f1338d87ca
      
https://github.com/qemu/qemu/commit/be9d199751789fdc96a3febe3f0768f1338d87ca
  Author: Peter Maydell <address@hidden>
  Date:   2017-10-03 (Tue, 03 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include
    M tests/docker/common.rc
    M tests/docker/dockerfiles/centos6.docker
    M tests/docker/dockerfiles/centos7.docker
    M tests/docker/dockerfiles/debian-ports.docker
    M tests/docker/dockerfiles/debian8.docker
    M tests/docker/dockerfiles/debian9.docker
    M tests/docker/dockerfiles/fedora.docker
    M tests/docker/dockerfiles/min-glib.docker
    M tests/docker/dockerfiles/ubuntu.docker
    M tests/docker/run
    M tests/docker/test-block
    M tests/docker/test-build
    M tests/docker/test-clang
    M tests/docker/test-full
    M tests/docker/test-mingw
    M tests/docker/test-quick

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/famz/tags/docker-testing-pull-request' 
into staging

# gpg: Signature made Fri 29 Sep 2017 04:17:37 BST
# gpg:                using RSA key 0xCA35624C6A9171C6
# gpg: Good signature from "Fam Zheng <address@hidden>"
# gpg: WARNING: This key is not certified with sufficiently trusted signatures!
# gpg:          It is not certain that the signature belongs to the owner.
# Primary key fingerprint: 5003 7CB7 9706 0F76 F021  AD56 CA35 624C 6A91 71C6

* remotes/famz/tags/docker-testing-pull-request:
  docker: Don't mount ccache db if NOUSER=1
  docker: test-block: Don't continue if build fails
  tests/docker/run: don't source /etc/profile
  docker: Fix test-mingw
  docker: add installation to build tests

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/ab1615292619...be9d19975178