qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] fab322: docker: Use stable git tag for virglr


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] fab322: docker: Use stable git tag for virglrenderer
Date: Tue, 15 Jan 2019 03:38:39 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: fab3220f973b6ee924bff691e15836baed22be3d
      
https://github.com/qemu/qemu/commit/fab3220f973b6ee924bff691e15836baed22be3d
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/debian-amd64.docker

  Log Message:
  -----------
  docker: Use stable git tag for virglrenderer

Use a stable tag instead of some random commit from mainstream
development, to avoid unexpected build failures.

This fixes:

    CC       virglrenderer.lo
  virglrenderer.c: In function 'virgl_has_gl_colorspace':
  virglrenderer.c:208:11: error: implicit declaration of function 
'virgl_has_egl_khr_gl_colorspace' [-Werror=implicit-function-declaration]
       virgl_has_egl_khr_gl_colorspace(egl_info));
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  virglrenderer.c:208:43: error: 'egl_info' undeclared (first use in this 
function)
       virgl_has_egl_khr_gl_colorspace(egl_info));
                                       ^~~~~~~~
  virglrenderer.c:208:43: note: each undeclared identifier is reported only 
once for each function it appears in
  cc1: some warnings being treated as errors

As of this commit 'git virglrenderer-0.7.0' is the last stable tag.
(virglrenderer commit breaking: fb4f7577f7ef)

Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 55923c7283f8c0f23dc837edd55e5c82067b2596
      
https://github.com/qemu/qemu/commit/55923c7283f8c0f23dc837edd55e5c82067b2596
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/debian-amd64.docker

  Log Message:
  -----------
  tests: run ldconfig after installing extra software

The docker file builds and installs software into /usr/local but does
not run ldconfig. As a result QEMU links to libvirglrenderer.so, but
then crashes in "make check" unable to find the library.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: ea08a74620e1df476fb7911a4e02565cd6cd8e69
      
https://github.com/qemu/qemu/commit/ea08a74620e1df476fb7911a4e02565cd6cd8e69
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    R tests/docker/dockerfiles/debian.docker

  Log Message:
  -----------
  tests: remove obsolete 'debian' dockerfile

The 'debian' dockerfile was deprecated in favour of versioned
dockerfiles in July 2017. That is enough time for developers to
be warned about the rename.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 19a9978db1aa434d5892e88ea9cae8f2b4cc1270
      
https://github.com/qemu/qemu/commit/19a9978db1aa434d5892e88ea9cae8f2b4cc1270
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/fedora.docker

  Log Message:
  -----------
  tests: update Fedora dockerfile to use Fedora 29

Fedora 29 is the current newest release, so switch to using that
from the current Fedora 28.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 62559b9167c5f3105867da36db88662df5ce135e
      
https://github.com/qemu/qemu/commit/62559b9167c5f3105867da36db88662df5ce135e
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/fedora-i386-cross.docker

  Log Message:
  -----------
  tests: update Fedora i386 cross image to Fedora 29

Using the "latest" tag is not a good idea because this changes what
release it points to every 6 months. Together with caching of docker
builds this can cause confusion where CI has cached & built with Fedora
N, while a developer tries to reproduce a CI problem with Fedora N + 1,
or vica-verca.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 53fe2431c77ead6d1a88cfeb43d557c34725309f
      
https://github.com/qemu/qemu/commit/53fe2431c77ead6d1a88cfeb43d557c34725309f
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/travis.docker

  Log Message:
  -----------
  tests: use in-place sed magic for enabling deb-src in travis image

This avoids potential problems with duplicates.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 197be697f8e011528055b5b5bdf056a405ae330f
      
https://github.com/qemu/qemu/commit/197be697f8e011528055b5b5bdf056a405ae330f
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: add whitespace between each major section & matrix entry

Improve the readability of the travis config by adding two blank lines
between each major section and matrix entry.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 3e094234fdda40e4ca8033bdf976d3da98437aae
      
https://github.com/qemu/qemu/commit/3e094234fdda40e4ca8033bdf976d3da98437aae
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: define all the build matrix entries in one place

The current build matrix is constructed from entries listed under the
environment variable config section, as well as the general purpose
build matrix section. Move everything under the general purpose section
so it is clear at a glance what is in the matrix.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: c21d7efc8bcd33db957958d75a9c0f5f93d62d77
      
https://github.com/qemu/qemu/commit/c21d7efc8bcd33db957958d75a9c0f5f93d62d77
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: standardize the syntax used for env variables

Matrix entries are defining env variables using two different syntax
styles:

   - env: FOO=bar
    WIZZ=bang

and

   - env:
       - FOO=bar
       - WIZZ=bang

Switch everything to use the latter style as the more normal indentation
approach.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 45ebc239f3eeb8231e7fd237ffb4d4ea86cbca97
      
https://github.com/qemu/qemu/commit/45ebc239f3eeb8231e7fd237ffb4d4ea86cbca97
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: don't clone git submodules upfront

The configure script & Makefile are already capable of figuring out
which git submodules are required for a given build platform, and
cloning them at the right time.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 91fa7dd100c9ce2d46e7cefddd935207e8a54ec7
      
https://github.com/qemu/qemu/commit/91fa7dd100c9ce2d46e7cefddd935207e8a54ec7
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: use homebrew addon for MacOSX

Rather than poking homebrew manually we can specify the packages
needed via the homebrew addon. These are only installed on MacOS based
builds.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: 9b29d0490d3f8b912d7a635004c89f9ef0a61b52
      
https://github.com/qemu/qemu/commit/9b29d0490d3f8b912d7a635004c89f9ef0a61b52
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: stop redefining the script commands

One of the matrix entries redefines the script command in order to add
the ${MAKEFLAGS} variable. Ideally ${MAKEFLAGS} would be referenced by
the definition of the ${TEST_CMD} env variable, but this isn't possible
in travis. ${MAKEFLAGS} exists to eliminate duplication of flags in
every "make" command, but this cure causes a worse problem, namely the
reduplication of the "script" command. It is simpler to just insert "-j3"
directly into any "make" command.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 4036878c3a02cb29291b8a1679a7e283f674fb3c
      
https://github.com/qemu/qemu/commit/4036878c3a02cb29291b8a1679a7e283f674fb3c
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: stop using container based envs

The Travis container based envs are deprecated:

  https://docs.travis-ci.com/user/reference/trusty/

  "Container-based infrastructure is currently being deprecated.
   Please remove any sudo: false keys in your .travis.yml file
   to use the default fully-virtualized Linux infrastructure
   instead."

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: e2576f9bf83486e3991ffa3fd47e34591f40f39f
      
https://github.com/qemu/qemu/commit/e2576f9bf83486e3991ffa3fd47e34591f40f39f
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: run tests in verbose mode

Travis sometimes fails a build because it produces no console output for
over 10 minutes. If this is due to a genuine hang, it would be useful to
have used verbose test output to see where it failed. If this is just
due to tests being very slow, having verbose output might allow the
build to succeed.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>


  Commit: 2e7908002e9dc7b89b5ee4179072a411a5ddb436
      
https://github.com/qemu/qemu/commit/2e7908002e9dc7b89b5ee4179072a411a5ddb436
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: remove matrix settings that duplicate global settings

The global defaults request "trusty" and "gcc", so matrix entries do not
need to repeat this.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: b4048a7cd10376412496c7241837f1a1d3b673fa
      
https://github.com/qemu/qemu/commit/b4048a7cd10376412496c7241837f1a1d3b673fa
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/debian-sid.docker

  Log Message:
  -----------
  docker: Use a stable snapshot for Debian Sid

The Debian Sid repository is not garanteed to be stable, as his
'unstable' name suggest :)
To allow quick testing, packages are pushed various time a day,
which my be annoying when trying to use it for stable development
(which is not recommended, but Sid provides edge packages we use
for testing).

Debian provides repositories snapshots which are suitable for our
use. Pick a recent date that works. When required, update to newer
releases will be easy.

This fixes current issues with this image:

  $ make docker-image-debian-sid
  [...]
  The following packages have unmet dependencies:
   build-essential : Depends: dpkg-dev (>= 1.17.11) but it is not going to be 
installed
   git : Depends: perl but it is not going to be installed
   Depends: liberror-perl but it is not going to be installed
   pkg-config : Depends: libdpkg-perl but it is not going to be installed
   texinfo : Depends: perl (>= 5.26.2-6) but it is not going to be installed
       Depends: libtext-unidecode-perl but it is not going to be installed
       Depends: libxml-libxml-perl but it is not going to be installed
  E: Unable to correct problems, you have held broken packages.

Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
[AJB: also tweak FROM to a earlier snapshot]
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 9531057670373743141fb780756510d817c4bcaa
      
https://github.com/qemu/qemu/commit/9531057670373743141fb780756510d817c4bcaa
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis: bump to Xenial baseline

Travis is slowly catching up. Move to Xenial based images for our
current builds. These are now all proper VMs.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: 3cc34204ea13db2bf89388189bc843171843eba0
      
https://github.com/qemu/qemu/commit/3cc34204ea13db2bf89388189bc843171843eba0
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/dockerfiles/travis.docker

  Log Message:
  -----------
  tests/docker: update our Travis image

We are now using Xenial based images on Travis so we should make the
same one available as our qemu:travis docker image.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: d406015b6a1e0cc8eda10816ddbecd7917f79297
      
https://github.com/qemu/qemu/commit/d406015b6a1e0cc8eda10816ddbecd7917f79297
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  tests/docker: remove SID_AGE test hack

Now we are using "named" snapshots of debian-sid we can rely on the
existing checksum mechanism for detecting changes.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: eb4f8e100f1ffce3be797ac56dcad031ffef3b8f
      
https://github.com/qemu/qemu/commit/eb4f8e100f1ffce3be797ac56dcad031ffef3b8f
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M tests/atomic64-bench.c
    M tests/atomic_add-bench.c
    M tests/qht-bench.c

  Log Message:
  -----------
  tests: use g_usleep instead of rem = sleep(time)

Relying on sleep to always return having slept isn't safe as a signal
may have occurred. If signals are constantly incoming the program will
never reach its termination condition. This is believed to be the
mechanism causing time outs for qht-test in Travis.

The glib g_usleep() deals with all of this for us so lets use it instead.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>


  Commit: a36270a4d1589b1ed309065fc8b3fe0ac8d6869d
      
https://github.com/qemu/qemu/commit/a36270a4d1589b1ed309065fc8b3fe0ac8d6869d
  Author: Alex Bennée <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M configure
    M tests/Makefile.include

  Log Message:
  -----------
  Revert "tests: Disable qht-bench parallel test when using gprof"

This reverts commit ce2eefd7c21697fee87a0686353de881081d22c6. The
underlying cause was fixed with eb4f8e100f.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: eb0667fe7f1b6d40dfb7172f17ab4db975acbbd6
      
https://github.com/qemu/qemu/commit/eb0667fe7f1b6d40dfb7172f17ab4db975acbbd6
  Author: Peter Maydell <address@hidden>
  Date:   2019-01-14 (Mon, 14 Jan 2019)

  Changed paths:
    M .travis.yml
    M configure
    M tests/Makefile.include
    M tests/atomic64-bench.c
    M tests/atomic_add-bench.c
    M tests/docker/Makefile.include
    M tests/docker/dockerfiles/debian-amd64.docker
    M tests/docker/dockerfiles/debian-sid.docker
    R tests/docker/dockerfiles/debian.docker
    M tests/docker/dockerfiles/fedora-i386-cross.docker
    M tests/docker/dockerfiles/fedora.docker
    M tests/docker/dockerfiles/travis.docker
    M tests/qht-bench.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/stsquad/tags/pull-testing-next-140119-1' into staging

A bunch of fixes for testing:

  - Various Travis updates
  - "stable" SID snapshot for docker
  - avoid :latest docker tags
  - g_usleep fix for some tests

# gpg: Signature made Mon 14 Jan 2019 14:59:35 GMT
# gpg:                using RSA key FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) <address@hidden>"
# Primary key fingerprint: 6685 AE99 E751 67BC AFC8  DF35 FBD0 DB09 5A9E 2A44

* remotes/stsquad/tags/pull-testing-next-140119-1: (21 commits)
  Revert "tests: Disable qht-bench parallel test when using gprof"
  tests: use g_usleep instead of rem = sleep(time)
  tests/docker: remove SID_AGE test hack
  tests/docker: update our Travis image
  travis: bump to Xenial baseline
  docker: Use a stable snapshot for Debian Sid
  travis: remove matrix settings that duplicate global settings
  travis: run tests in verbose mode
  travis: stop using container based envs
  travis: stop redefining the script commands
  travis: use homebrew addon for MacOSX
  travis: don't clone git submodules upfront
  travis: standardize the syntax used for env variables
  travis: define all the build matrix entries in one place
  travis: add whitespace between each major section & matrix entry
  tests: use in-place sed magic for enabling deb-src in travis image
  tests: update Fedora i386 cross image to Fedora 29
  tests: update Fedora dockerfile to use Fedora 29
  tests: remove obsolete 'debian' dockerfile
  tests: run ldconfig after installing extra software
  ...

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


Compare: https://github.com/qemu/qemu/compare/89bd861c2b47...eb0667fe7f1b
      **NOTE:** This service has been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]