Version: libvirt/6.0.0-1 Hi, marking as fixed too. -- Guido
On Tue, Aug 07, 2018 at 08:13:51AM +0200, Christian Ehrhardt wrote: > Package: libvirt > Version: 4.6.0-1 > Severity: normal > > Debian has taken: > * [d53b4b1] Use jansson instead of yajl. The later is no longer supported > upstream > Which is following upsteram fine. > > But Upstream: ce3c6ef6 util: avoid symbol clash between json libraries > Had: > --- a/libvirt.spec.in > +++ b/libvirt.spec.in > @@ -898,6 +898,8 @@ Requires: ncurses > Requires: gettext > # Needed by virt-pki-validate script. > Requires: gnutls-utils > +# We dlopen(libjansson.so.4), so need an explicit dep > +Requires: jansson > %if %{with_bash_completion} > Requires: %{name}-bash-completion = %{version}-%{release} > %endif > > This is special since this is intentionally "black dlopen magic" to avoid > the issues around the symbol collision that was discussed on -rc2. > > Build log [1] and a debian-sid container show it as missing. > So you could end up with a dlopen to a nonexisting file. > This will not be found by shlibs and co. > > I tested on sid if there is a hidden > > Fix looks like a normal dependency add. > I'm slightly unsure where exactly, but the old libyajl2 dependencies were > on libvirt0, libvirt-daemon and libnss-libvirt which is I think the set we > should go for. > > [1]: > https://buildd.debian.org/status/fetch.php?pkg=libvirt&arch=amd64&ver=4.6.0-1&stamp=1533592540&raw=0 > > -- > Christian Ehrhardt > Software Engineer, Ubuntu Server > Canonical Ltd