[PULL,01/21] docker: Use stable git tag for virglrenderer

Message ID	20190114150129.1013-2-alex.bennee@linaro.org
State	Accepted
Commit	fab3220f973b6ee924bff691e15836baed22be3d
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; From: =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org> To: peter.maydell@linaro.org Date: Mon, 14 Jan 2019 15:01:09 +0000 Message-Id: <20190114150129.1013-2-alex.bennee@linaro.org> In-Reply-To: <20190114150129.1013-1-alex.bennee@linaro.org> References: <20190114150129.1013-1-alex.bennee@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PULL 01/21] docker: Use stable git tag for virglrenderer Precedence: list Cc: Fam Zheng <fam@euphon.net>, =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org>, =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= <philmd@redhat.com>, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>
Series	misc testing fixes for Travis and docker \| expand [PULL,00/21] misc testing fixes for Travis and docker [PULL,01/21] docker: Use stable git tag for virglrenderer [PULL,02/21] tests: run ldconfig after installing extra software [PULL,03/21] tests: remove obsolete 'debian' dockerfile [PULL,04/21] tests: update Fedora dockerfile to use Fedora 29 [PULL,05/21] tests: update Fedora i386 cross image to Fedora 29 [PULL,06/21] tests: use in-place sed magic for enabling deb-src in travis image [PULL,07/21] travis: add whitespace between each major section & matrix entry [PULL,08/21] travis: define all the build matrix entries in one place [PULL,09/21] travis: standardize the syntax used for env variables [PULL,10/21] travis: don't clone git submodules upfront [PULL,11/21] travis: use homebrew addon for MacOSX [PULL,12/21] travis: stop redefining the script commands [PULL,13/21] travis: stop using container based envs [PULL,14/21] travis: run tests in verbose mode [PULL,15/21] travis: remove matrix settings that duplicate global settings [PULL,16/21] docker: Use a stable snapshot for Debian Sid [PULL,17/21] travis: bump to Xenial baseline [PULL,18/21] tests/docker: update our Travis image [PULL,19/21] tests/docker: remove SID_AGE test hack [PULL,20/21] tests: use g_usleep instead of rem = sleep(time) [PULL,21/21] Revert "tests: Disable qht-bench parallel test when using gprof"

Message ID

20190114150129.1013-2-alex.bennee@linaro.org

State

Accepted

Commit

fab3220f973b6ee924bff691e15836baed22be3d

Headers

Received-SPF: pass (google.com: domain of
	qemu-devel-bounces+patch=linaro.org@nongnu.org designates
	209.51.188.17 as permitted sender) client-ip=209.51.188.17; 
From: =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org>
To: peter.maydell@linaro.org
Date: Mon, 14 Jan 2019 15:01:09 +0000
Message-Id: <20190114150129.1013-2-alex.bennee@linaro.org>
In-Reply-To: <20190114150129.1013-1-alex.bennee@linaro.org>
References: <20190114150129.1013-1-alex.bennee@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: [Qemu-devel] [PULL 01/21] docker: Use stable git tag for
	virglrenderer
Precedence: list
Cc: Fam Zheng <fam@euphon.net>, =?utf-8?q?Alex_Benn=C3=A9e?=
	<alex.bennee@linaro.org>, =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?=
	<philmd@redhat.com>, 	qemu-devel@nongnu.org
Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org
Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>

Series

misc testing fixes for Travis and docker | expand

Commit Message

Alex Bennée Jan. 14, 2019, 3:01 p.m. UTC

From: Philippe Mathieu-Daudé <philmd@redhat.com>


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é <philmd@redhat.com>

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


-- 
2.17.1

diff --git a/tests/docker/dockerfiles/debian-amd64.docker b/tests/docker/dockerfiles/debian-amd64.docker
index 24b113b76f..47a30adbdb 100644
--- a/tests/docker/dockerfiles/debian-amd64.docker
+++ b/tests/docker/dockerfiles/debian-amd64.docker
@@ -24,7 +24,8 @@  RUN DEBIAN_FRONTEND=noninteractive eatmydata \
         libegl1-mesa-dev \
         libepoxy-dev \
         libgbm-dev
-RUN git clone https://anongit.freedesktop.org/git/virglrenderer.git /usr/src/virglrenderer
+RUN git clone https://anongit.freedesktop.org/git/virglrenderer.git /usr/src/virglrenderer && \
+    cd /usr/src/virglrenderer && git checkout virglrenderer-0.7.0
 RUN cd /usr/src/virglrenderer && ./autogen.sh && ./configure --with-glx --disable-tests && make install
 
 # netmap

[PULL,01/21] docker: Use stable git tag for virglrenderer

Commit Message

Patch