From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36732) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLia2-0006Xj-Ko for qemu-devel@nongnu.org; Tue, 19 Jan 2016 21:33:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aLiZy-00032P-OY for qemu-devel@nongnu.org; Tue, 19 Jan 2016 21:33:06 -0500 Received: from ozlabs.org ([103.22.144.67]:52022) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLiZy-00031p-Dm for qemu-devel@nongnu.org; Tue, 19 Jan 2016 21:33:02 -0500 Date: Wed, 20 Jan 2016 13:29:20 +1100 From: David Gibson Message-ID: <20160120022920.GI27454@voom.redhat.com> References: <1452854709-12855-1-git-send-email-alex.bennee@linaro.org> <20160119061505.GC27454@voom.redhat.com> <87fuxuc9uo.fsf@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="xGGVyNQdqA79rdfn" Content-Disposition: inline In-Reply-To: <87fuxuc9uo.fsf@linaro.org> Subject: Re: [Qemu-devel] [PATCH] .travis.yml: migrate to container builds List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex =?iso-8859-1?Q?Benn=E9e?= Cc: pbonzini@redhat.com, qemu-devel@nongnu.org --xGGVyNQdqA79rdfn Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 19, 2016 at 07:22:07AM +0000, Alex Benn=E9e wrote: >=20 > David Gibson writes: >=20 > > On Fri, Jan 15, 2016 at 10:45:09AM +0000, Alex Benn=E9e wrote: > >> This moves the Travis tests from their old legacy VM > >> infrastructure (which only seems to run 5-6 jobs at once) to their new > >> container based approach. > >> > >> The principle difference is there is no sudo in the containers so all > >> packages are installed using the apt add-on. This means one of the bui= ld > >> combinations can be dropped as it was only for checking the build with > >> additional packages. > >> > >> I've disabled the user-space tracing build until the dependant packages > >> go through the Travis package white-listing process. > >> > >> Signed-off-by: Alex Benn=E9e > > > > Tested-by: David Gibson > > > > Is it safe to just drop that build combination, or should we instead > > be doing something else to test the build _without_ those extra > > packages? >=20 > Hmm, currently there doesn't seem to be a way to do this with the Travis > infrastructure as the packages are no longer part of a matrix. Ok, but would it work to explicitly disable use of the packages in some of the builds? > > [snip] > >> @@ -86,10 +103,10 @@ matrix: > >> - env: TARGETS=3Di386-softmmu,x86_64-softmmu > >> EXTRA_CONFIG=3D"--enable-trace-backends=3Dftrace" > >> compiler: gcc > >> - - env: TARGETS=3Di386-softmmu,x86_64-softmmu > >> - EXTRA_PKGS=3D"liblttng-ust-dev liburcu-dev" > >> - EXTRA_CONFIG=3D"--enable-trace-backends=3Dust" > >> - compiler: gcc > >> + # UST backend disabled until liblttng-ust-dev pkg white-listed > >> + #- env: TARGETS=3Di386-softmmu,x86_64-softmmu > >> + # EXTRA_CONFIG=3D"--enable-trace-backends=3Dust" > >> + # compiler: gcc > > > > This comment is a bit confusing to me, since the apt addon package > > list already seems to include liblttng-ust-dev. >=20 > It's in the list but still awaiting whitelisting by Travis: >=20 > https://github.com/travis-ci/apt-package-whitelist/issues/2258 Ah, ok. I'm kind of surprised that Travis passes a build requesting a non-whitelisted package. Do we want to wait until that's approved before merging? I don't have a strong opinion either way. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --xGGVyNQdqA79rdfn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWnvEAAAoJEGw4ysog2bOSBnwQAL3mQ0lHW9IxKgGZJguQ12t2 qTEHw6w65iGrX1dyNTQNrLcHzpMDuMxGlDzd5So1EbXsuIn9jFyJZSgvpK8iRH3X IAGn6dbR2pDbqv4jraRZCiKSSLjJlqDY+gy6MxdphjMBR2YZ33c/HlZvCKEK8EuU IsbZaE46JKai2wnUYkW+vU7XeF4clRzbPwqE0eS699q+g9EQpS10fSQcp99+VVOL ULci5QcAwrVqMlrqMw03Cx9qR4pdpRXJa16iKsmml/7bEeck8coCsA27gr1k1V+X OXZ+zqM2f1t1G0Tfd0+koQ7QBzBI6EJZ6c7/Z9ouetkZ7BecmR4ucVmNr9vOHTNP qWpoq0P2gvUovSxX5fMS3ft3Qoq9ieRvwfU68Uo8S1qKN8ByE1F8hpoggoLRKPpK 7fcTPfnFIIa/CuLsMMMuRVwX5XLfu+N+4lUsojRCAjtSZRSBbp5UNJWdZ2o67sg2 s2ff5dWdAOuBEA7SrznliI7RRZKvT8t9w8y7Jet/WyVlocKgsxXo8NLFyz/YF591 Hpz5RXlxDmhurlRZpSgjWHeDiJH/wjsMGGknqnFbh2jAqeqBxe+8dzdnt/iGC0So 7assnQYsmWOKIBUARSr5koA/62rhOEpDnHGMqbyo5XLjg5mk1L7beEIbtr5u4oSq rCYSm9jzHuuaaubRed4U =V7Rv -----END PGP SIGNATURE----- --xGGVyNQdqA79rdfn--