[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/6] travis.yml: Update Travis to use Bionic and Focal instea
From: |
Thomas Huth |
Subject: |
Re: [PATCH 5/6] travis.yml: Update Travis to use Bionic and Focal instead of Xenial |
Date: |
Tue, 22 Sep 2020 08:55:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 22/09/2020 01.39, Cleber Rosa wrote:
> On Fri, Sep 18, 2020 at 12:34:29PM +0200, Thomas Huth wrote:
>> According to our support policy, we do not support Xenial anymore.
>> Time to switch the bigger parts of the builds to Focal instead.
>> Some few jobs have to be updated to Bionic instead, since they are
>> currently still failing on Focal otherwise. Also "--disable-pie" is
>> causing linker problems with newer versions of Ubuntu ... so remove
>> that switch from the jobs now (we still test it in a gitlab CI job,
>> so we don't lose much test coverage here).
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> .travis.yml | 24 +++++++++++++-----------
>> 1 file changed, 13 insertions(+), 11 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index 1fc49b0746..80da4ebc8e 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -2,7 +2,7 @@
>> # Additional builds with specific requirements for a full VM need to
>> # be added as additional matrix: entries later on
>> os: linux
>> -dist: xenial
>> +dist: focal
>> language: c
>> compiler:
>> - gcc
>> @@ -10,7 +10,7 @@ cache:
>> # There is one cache per branch and compiler version.
>> # characteristics of each job are used to identify the cache:
>> # - OS name (currently only linux)
>> - # - OS distribution (for Linux, xenial, trusty, or precise)
>> + # - OS distribution (for Linux, bionic or focal)
>> # - Names and values of visible environment variables set in .travis.yml
>> or Settings panel
>> timeout: 1200
>> ccache: true
>> @@ -27,7 +27,7 @@ addons:
>> - libattr1-dev
>> - libbrlapi-dev
>> - libcap-ng-dev
>> - - libgcc-4.8-dev
>> + - libgcc-7-dev
>> - libgnutls28-dev
>> - libgtk-3-dev
>> - libiscsi-dev
>> @@ -211,8 +211,10 @@ jobs:
>>
>> # gprof/gcov are GCC features
>> - name: "GCC gprof/gcov"
>> + dist: bionic
>> env:
>> - - CONFIG="--enable-gprof --enable-gcov --disable-pie
>> --target-list=${MAIN_SOFTMMU_TARGETS}"
>> + - CONFIG="--enable-gprof --enable-gcov --disable-libssh
>> + --target-list=${MAIN_SOFTMMU_TARGETS}"
>
> While the commit message describes the reason for dropping
> '--disable-pie', it doesn't talk about why it adds '--disable-libssh'.
Ah, well, right. I should have mentioned that... but we've hit that
problem in the past so often that my mind apparently sorted it into the
well-known category and did not came up with the idea that it would be
good to mention it ;-)
The problem is that libssh is very much broken in Bionic. Ubuntu
backported some patches into that version so that QEMU's configure
script thinks that it is a newer version, but they did not backport some
other important patches so that compilation later fails. Since it's
rather a bug in the distro and not in QEMU, we never "fixed" this issue,
so libssh can not be used in Bionic. See also:
https://bugs.launchpad.net/qemu/+bug/1838763
> Reviewed-by: Cleber Rosa <crosa@redhat.com>
> Tested-by: Cleber Rosa <crosa@redhat.com>
Thanks!
Thomas
- [PATCH 0/6] Update Travis from Xenial to Bionic and Focal, Thomas Huth, 2020/09/18
- [PATCH 1/6] meson: move libudev test, Thomas Huth, 2020/09/18
- [PATCH 2/6] meson: move libmpathpersist test, Thomas Huth, 2020/09/18
- [PATCH 5/6] travis.yml: Update Travis to use Bionic and Focal instead of Xenial, Thomas Huth, 2020/09/18
- [PATCH 4/6] travis.yml: Drop the default softmmu builds, Thomas Huth, 2020/09/18
- [PATCH 3/6] migration: Silence compiler warning in global_state_store_running(), Thomas Huth, 2020/09/18
- [PATCH 6/6] travis.yml: Drop the superfluous Python 3.6 build, Thomas Huth, 2020/09/18
- Re: [PATCH 0/6] Update Travis from Xenial to Bionic and Focal, Paolo Bonzini, 2020/09/18
- Re: [PATCH 0/6] Update Travis from Xenial to Bionic and Focal, Alex Bennée, 2020/09/21