[libvirt] [jenkins-ci PATCH 2/6] mingw*-libosinfo: Add mingw*-libsoup as dependency

Andrea Bolognani abologna at redhat.com
Wed Jul 31 12:37:37 UTC 2019


On Wed, 2019-07-31 at 14:24 +0200, Fabiano Fidêncio wrote:
> On Wed, Jul 31, 2019 at 1:54 PM Andrea Bolognani <abologna at redhat.com> wrote:
> > On Mon, 2019-07-29 at 10:41 +0200, Fabiano Fidêncio wrote:
> > > It's been introduced as dependency as part of v1.6.0 release.
> > 
> > Is that so? Because the native build has had the dependency on
> > libsoup ever since
> > 
> >   commit 1eab5a66eb0abd922a67e2c96a4c46efce525548
> >   Author: Andrea Bolognani <abologna at redhat.com>
> >   Date:   Tue Oct 3 18:11:38 2017 +0200
> > 
> >     ansible: Add libosinfo project
> > 
> >     Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> > 
> > eg. the day it was introduced. So it seems to me we just messed
> > up when in
> 
> Old tests on libosinfo were relying on libsoup and that's the reason I
> do believe the dependency was there.
> Pino dropped the dependency a long time ago here:
> 
> ```
> commit b61be0405f678986ac344f414fa907af24670488
> Author:     Pino Toscano <ptoscano at redhat.com>
> AuthorDate: Mon Aug 7 18:18:34 2017 +0200
> Commit:     Christophe Fergeau <cfergeau at redhat.com>
> CommitDate: Mon Oct 16 11:12:22 2017 +0200
> 
>     Switch from libsoup to libcurl
> 
>     libsoup is used to check the validity of URLs in distributions in
>     osinfo-db; OTOH it supports only HTTP(S), so this limits the checks to
>     that protocol.
> 
>     To overcome this limitation, switch to libcurl: while it requires
>     slightly more code to do the same task, it provides a bit more
>     flexibility, and support for other protocols.  No version check is
>     performed, since the APIs used are old enough.
> 
>     Adapt also the README, and the packaging files.
> ```
> However, libvirt-jenkins-ci was never updated.

Alright, I'll keep your commit message as is but I'll add

  The native build already depended on libsoup because it was never
  updated after libosinfo switched its test suite from libsoup to
  libcurl back in 2017.

and then push the series.

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list