Your message dated Fri, 25 Nov 2022 14:58:30 +0100
with message-id <y4dkbo7ahcypj...@qwark.sigxcpu.org>
and subject line Re: Bug#905613: jansson dependency needs to be added explcitly
has caused the Debian Bug report #905613,
regarding jansson dependency needs to be added explcitly
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
905613: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=905613
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
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
--- End Message ---
--- Begin Message ---
Version: libvirt/7.0.0-3
Hi,
On Fri, Apr 03, 2020 at 10:30:17AM +0200, Guido Günther wrote:
> Version: libvirt/6.0.0-1
>
> Hi,
> marking as fixed too.
Closing as this wasn't done via the changelog.
-- Guido
> -- 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
--- End Message ---
