[Qemu-commits] [qemu/qemu] 4b2c6b: docker: Don't allocate tty unless DEBUG=1

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 4b2c6bc563c13f242d92fe4f8f31d22b31788858
      
https://github.com/qemu/qemu/commit/4b2c6bc563c13f242d92fe4f8f31d22b31788858
  Author: Fam Zheng <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  docker: Don't allocate tty unless DEBUG=1

The existence of tty in the container seems to urge gcc into colorizing
the errors, but the escape chars will clutter the report once turned
into email replies on patchew. Move -t to debug mode.

Reported-by: Eric Blake <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 08c3f6d57f9a95036eed1eea0571eb47605aaef4
      
https://github.com/qemu/qemu/commit/08c3f6d57f9a95036eed1eea0571eb47605aaef4
  Author: Peter Xu <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  docker: cleanup temp directory after test

There are temp directories named "docker-src.*" after doing docker
tests.  I don't see much point in keeping that (it only contains the
qemu.tar which is exactly current tree, and the copied "run" file).
Let's remove it after test finished.

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


  Commit: bc739b8e7d02c8010a5ab0a5f4d6ed99c1bb1d37
      
https://github.com/qemu/qemu/commit/bc739b8e7d02c8010a5ab0a5f4d6ed99c1bb1d37
  Author: Paolo Bonzini <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  docker: allow running from srcdir != builddir build

The new script uses "git submodule", which is picky about being invoked
from the top of the git checkout.  Invoke the script from $(SRC_PATH)
to avoid git's wrath.

Fixes: b7f404201e45e99da23b9764dec27ce5f965cdcd
Cc: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>

Message-Id: <address@hidden>
Reviewed-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 1dc1700a60ece53c77d0e396c92b5acff9d66bf2
      
https://github.com/qemu/qemu/commit/1dc1700a60ece53c77d0e396c92b5acff9d66bf2
  Author: Greg Kurz <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  docker: fix out-of-tree 'make address@hidden'

Without this patch, it fails with:

IOError: [Errno 2] No such file or directory:
 'tests/docker/dockerfiles/debian-apt-fake.sh'

Signed-off-by: Greg Kurz <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Fam Zheng <address@hidden>


  Commit: 6a2e11974fa6c4eaee6b2b156b1b4e03ca983bab
      
https://github.com/qemu/qemu/commit/6a2e11974fa6c4eaee6b2b156b1b4e03ca983bab
  Author: Fam Zheng <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/run

  Log Message:
  -----------
  docker: Fix PATH for ccache

Before bcd7f06f57fb6f780a3e2f7a46c22b6f6c8238aa we source /etc/profile
so the PATH included the right paths to ccache binaries. Now we need to
update $PATH explicitly from run script.

Keep the old /usr/lib around just so that in the future, ccache from 32
bit images will just work.

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


  Commit: 27825bc954f832a8e1137ceb45b52f7bb3f8f9cd
      
https://github.com/qemu/qemu/commit/27825bc954f832a8e1137ceb45b52f7bb3f8f9cd
  Author: Peter Maydell <address@hidden>
  Date:   2017-10-20 (Fri, 20 Oct 2017)

  Changed paths:
    M tests/docker/Makefile.include
    M tests/docker/run

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

# gpg: Signature made Fri 20 Oct 2017 07:30:45 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-pull-request:
  docker: Fix PATH for ccache
  docker: fix out-of-tree 'make address@hidden'
  docker: allow running from srcdir != builddir build
  docker: cleanup temp directory after test
  docker: Don't allocate tty unless DEBUG=1

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


Compare: https://github.com/qemu/qemu/compare/e67277f8f32d...27825bc954f8