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(a)redhat.com&gt; Date: Tue Oct 3 18:11:38 2017 +0200 ansible: Add libosinfo project Signed-off-by: Andrea Bolognani <abologna(a)redhat.com&gt; eg. the day it was introduced. So it seems to me we just messed up when in

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(a)redhat.com&gt; 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(a)redhat.com&gt; > > Date: Tue Oct 3 18:11:38 2017 +0200 > > > > ansible: Add libosinfo project > > > > Signed-off-by: Andrea Bolognani <abologna(a)redhat.com&gt; > > > > 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(a)redhat.com&gt; > AuthorDate: Mon Aug 7 18:18:34 2017 +0200 > Commit: Christophe Fergeau <cfergeau(a)redhat.com&gt; > 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.

On Mon, 2019-07-29 at 10:41 +0200, Fabiano Fidêncio wrote: > libsoup & mingw-libsoup are new dependencies for both libosinfo & > osinfo-db-tools since v1.6.0 release > libcurl & mingw-curl are no longer a dependency for libosinfo since > v1.5.0 release > > Fabiano Fidêncio (6): > mappings: Add mingw*-libsoup > mingw*-libosinfo: Add mingw*-libsoup as dependency > mingw*-osinfo-db-tools: Add mingw*-libsoup as dependency > osinfo-db-tools: Add libsoup as dependency > mingw*-libosinfo: Remove mingw*-curl dependency > ibosinfo: Remove libcurl dependency Aside from the typo in 1/6 Dan already pointed out and the question of the commit message of 2/6 not being AFAICT accurate, one more meta-comment is that 3-4/6 and 5-6/6 should be squashed together: I have already done so locally, so as soon as you clear up the situation for 2/6 I can push the entire series with my Reviewed-by: Andrea Bolognani <abologna(a)redhat.com&gt;