a questionable dependency chain

[Matthias Clasen]

I've switched to rawhide yesterday, and discovered that vinagre now
forces rsyslog onto my system. That's not great.

The dependency chain goes something like this:

vinagre -> spice -> libcacard -> ... glusterfs ... ->
rsyslog-mmjsonparse -> rsyslog

I think there's at least two questionable links in this chain:

- why does libcacard need glusterfs ?

- why does glusterfs need rsyslog ?

Can we get one or both of these cut, maybe ?

Matthias

[Daniel P. Berrange]

On Wed, Jan 29, 2014 at 09:21:22AM -0500, Matthias Clasen wrote:
>  I've switched to rawhide yesterday, and discovered that vinagre
> now
>  forces rsyslog onto my system. That's not great.
>  
>  The dependency chain goes something like this:
>  
>  vinagre -> spice -> libcacard -> ... glusterfs ... ->
>  rsyslog-mmjsonparse -> rsyslog
>  
>  I think there's at least two questionable links in this chain:
>  
>  - why does libcacard need glusterfs ? 
libcacard is built as part of QEMU source code. QEMU needs glusterfs
but  libcacard should not need it. I'd guess that QEMU's hand-crafted
makefiles have fubar'd the linker args, adding '-lglusterfs' to the
libcacard build, despite it not being used. Best file a bug against
QEMU for this, as it should definitely be fixed.

>  - why does glusterfs need rsyslog ? 
Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

[Christophe Fergeau]

Hey,

On Wed, Jan 29, 2014 at 09:21:22AM -0500, Matthias Clasen wrote:
>  I've switched to rawhide yesterday, and discovered that vinagre
> now
>  forces rsyslog onto my system. That's not great.
>  
>  The dependency chain goes something like this:
>  
>  vinagre -> spice -> libcacard -> ... glusterfs ... ->
>  rsyslog-mmjsonparse -> rsyslog
>  
>  I think there's at least two questionable links in this chain:
>  
>  - why does libcacard need glusterfs ? 
Looking at nm output for libcacard.so, I did not see symbols that were
obviously related to glusterfs there. However, at build time, libcacard
ld command line gets a lot of -lxx, including glusterfs libraries.
libcacard is a sub-package of qemu, so I suspect what's happening is
it gets linked with all the libraries qemu needs.

This patch seems to help there and get rid of the extra libraries in ldd
output (did not try an rpm build to check the deps there).

diff --git a/libcacard/Makefile b/libcacard/Makefile
index 47827a0..9fa297c 100644
--- a/libcacard/Makefile
+++ b/libcacard/Makefile
@@ -24,7 +24,7 @@ vscclient$(EXESUF): libcacard/vscclient.o libcacard.la

 libcacard.la: LDFLAGS += -rpath $(libdir) -no-undefined \
        -export-syms $(SRC_PATH)/libcacard/libcacard.syms
-libcacard.la: LIBS += $(libcacard_libs)
+libcacard.la: LIBS = $(libcacard_libs)
 libcacard.la: $(libcacard-lobj-y)
        $(call LINK,$^)


Christophe

[Orion Poplawski]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/29/2014 07:32 AM, Christophe Fergeau wrote:
>  Hey,
>  
>  On Wed, Jan 29, 2014 at 09:21:22AM -0500, Matthias Clasen wrote:
> > I've switched to rawhide yesterday, and discovered that vinagre now 
> > forces rsyslog onto my system. That's not great.
> > 
> > The dependency chain goes something like this:
> > 
> > vinagre -> spice -> libcacard -> ... glusterfs ... -> 
> > rsyslog-mmjsonparse -> rsyslog
> > 
> > I think there's at least two questionable links in this chain:
> > 
> > - why does libcacard need glusterfs ?
>  
>  Looking at nm output for libcacard.so, I did not see symbols that were 
>  obviously related to glusterfs there. However, at build time, libcacard 
>  ld command line gets a lot of -lxx, including glusterfs libraries. 
>  libcacard is a sub-package of qemu, so I suspect what's happening is it
>  gets linked with all the libraries qemu needs.
>  
>  This patch seems to help there and get rid of the extra libraries in ldd 
>  output (did not try an rpm build to check the deps there).
>  
>  diff --git a/libcacard/Makefile b/libcacard/Makefile index
>  47827a0..9fa297c 100644 --- a/libcacard/Makefile +++
>  b/libcacard/Makefile @@ -24,7 +24,7 @@ vscclient$(EXESUF):
>  libcacard/vscclient.o libcacard.la
>  
>  libcacard.la: LDFLAGS += -rpath $(libdir) -no-undefined \ -export-syms
>  $(SRC_PATH)/libcacard/libcacard.syms -libcacard.la: LIBS +=
>  $(libcacard_libs) +libcacard.la: LIBS = $(libcacard_libs) libcacard.la:
>  $(libcacard-lobj-y) $(call LINK,$^)
>  
>  
>  Christophe 
Yeah, there were lots:

# rpmlint libcacard
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /usr/lib64/iscsi/libiscsi.so.2
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libz.so.1
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libgfapi.so.0
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libglusterfs.so.0
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libgfrpc.so.0
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libgfxdr.so.0
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libssh2.so.1
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/librbd.so.1
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/librados.so.2
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libcurl.so.4
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libuuid.so.1
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libaio.so.1
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libssl3.so
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libsmime3.so
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libnssutil3.so
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libplds4.so
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libplc4.so
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libdl.so.2
libcacard.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libcacard.so.0.0.0 /lib64/libgthread-2.0.so.0
libcacard.x86_64: W: shared-lib-calls-exit /usr/lib64/libcacard.so.0.0.0
exit(a)GLIBC_2.2.5
libcacard.x86_64: W: no-documentation

Also:

libcacard.x86_64: E: library-without-ldconfig-postin
/usr/lib64/libcacard.so.0.0.0
libcacard.x86_64: E: library-without-ldconfig-postun
/usr/lib64/libcacard.so.0.0.0

needs some scripts.

- -- 
Orion Poplawski
Technical Manager                     303-415-9701 x222
NWRA, Boulder/CoRA Office             FAX: [phone]
3380 Mitchell Lane                       orion(a)nwra.com
Boulder, CO 80301                   http://www.nwra.com
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
>
> iEYEARECAAYFAlLpUNIACgkQORnzrtFC2/ueVQCgnz2PbI0W4VF82eC/1D5Ov/9v
> aPYAn0roZC262UGIFePp+drX+dLFGpbG
> =HoH5
> -----END PGP SIGNATURE-----

[Ville Skyttä]

On Wed, Jan 29, 2014 at 9:04 PM, Orion Poplawski <orion(a)cora.nwra.com&gt; wrote:
>  Also:
>
>  libcacard.x86_64: E: library-without-ldconfig-postin
>  /usr/lib64/libcacard.so.0.0.0
>  libcacard.x86_64: E: library-without-ldconfig-postun
>  /usr/lib64/libcacard.so.0.0.0
>
>  needs some scripts. 
Already fixed in December, I suppose your rpmlint run wasn't against
the latest Rawhide libcacard?

http://pkgs.fedoraproject.org/cgit/qemu.git/commit/?id=c4896d008b4e71e0cd...

[Orion Poplawski]

On 01/29/2014 01:36 PM, Ville Skyttä wrote:
>  On Wed, Jan 29, 2014 at 9:04 PM, Orion Poplawski
> <orion(a)cora.nwra.com&gt; wrote:
> > Also:
> >
> > libcacard.x86_64: E: library-without-ldconfig-postin
> > /usr/lib64/libcacard.so.0.0.0
> > libcacard.x86_64: E: library-without-ldconfig-postun
> > /usr/lib64/libcacard.so.0.0.0
> >
> > needs some scripts.
>  
>  Already fixed in December, I suppose your rpmlint run wasn't against
>  the latest Rawhide libcacard?
>  
>
> http://pkgs.fedoraproject.org/cgit/qemu.git/commit/?id=c4896d008b4e71e0cd...
>   
Ah, oops.

-- 
Orion Poplawski
Technical Manager                     303-415-9701 x222
NWRA, Boulder/CoRA Office             FAX: [phone]
3380 Mitchell Lane                       orion(a)nwra.com
Boulder, CO 80301                   http://www.nwra.com

[Cole Robinson]

On 01/29/2014 09:32 AM, Christophe Fergeau wrote:
>  Hey,
>  
>  On Wed, Jan 29, 2014 at 09:21:22AM -0500, Matthias Clasen wrote:
> > I've switched to rawhide yesterday, and discovered that vinagre now
> > forces rsyslog onto my system. That's not great.
> >
> > The dependency chain goes something like this:
> >
> > vinagre -> spice -> libcacard -> ... glusterfs ... ->
> > rsyslog-mmjsonparse -> rsyslog
> >
> > I think there's at least two questionable links in this chain:
> >
> > - why does libcacard need glusterfs ?
>   
This is https://bugzilla.redhat.com/show_bug.cgi?id=987441

>  Looking at nm output for libcacard.so, I did not see symbols that
> were
>  obviously related to glusterfs there. However, at build time, libcacard
>  ld command line gets a lot of -lxx, including glusterfs libraries.
>  libcacard is a sub-package of qemu, so I suspect what's happening is
>  it gets linked with all the libraries qemu needs.
>  
>  This patch seems to help there and get rid of the extra libraries in ldd
>  output (did not try an rpm build to check the deps there).
>  
>  diff --git a/libcacard/Makefile b/libcacard/Makefile
>  index 47827a0..9fa297c 100644
>  --- a/libcacard/Makefile
>  +++ b/libcacard/Makefile
>  @@ -24,7 +24,7 @@ vscclient$(EXESUF): libcacard/vscclient.o libcacard.la
>  
>   libcacard.la: LDFLAGS += -rpath $(libdir) -no-undefined \
>          -export-syms $(SRC_PATH)/libcacard/libcacard.syms
>  -libcacard.la: LIBS += $(libcacard_libs)
>  +libcacard.la: LIBS = $(libcacard_libs)
>   libcacard.la: $(libcacard-lobj-y)
>          $(call LINK,$^)
>   
Thanks Christophe! Can you (or elmarco, or alevy), forward that to qemu-devel?

- Cole

[Christophe Fergeau]

On Thu, Jan 30, 2014 at 08:30:36AM -0500, Cole Robinson wrote:
>  Thanks Christophe! Can you (or elmarco, or alevy), forward that to
> qemu-devel? 
Done:
https://lists.nongnu.org/archive/html/qemu-devel/2014-01/msg04164.html

Christophe