Re: While we're talking about RPM dependencies ...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 05, 2012 at 08:21:15PM +0200, drago01 wrote:
> On Thu, Apr 5, 2012 at 8:03 PM, Richard W.M. Jones <rjones@xxxxxxxxxx> wrote:
> > Today when using F17 Alpha, I ran qemu and got an error which was
> > something like:
> >
> >  qemu-kvm: undefined symbol usbredirhost_foo
> >
> > (I don't recall the precise symbol).  This was just because that
> > version of qemu was compiled against a later version of
> > libusbredirhost.so (but one with the same soname), and updating
> > libusbredirhost.so fixed the problem.
> >
> > The 'qemu' package has a bug, of sorts: the maintainer should have
> > added a specific Requires line:
> >
> >  Requires: usbredir >= <some version>
> >
> > However, instead of pushing this problem on packagers, maybe RPM
> > should resolve this by encoding the (admittedly long) list of symbols
> > used by a binary?
> 
> Well the bug here is that usbredir changed ABI without bumping the soname.

They don't need to, nor should they, bump the soname when adding a
new symbol.  If they used symbol versioning, however, then RPM would
pick up the deps correctly, because it adds deps on all ELF symbol
versions


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 :|
-- 
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/devel



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux