[libvirt] [PATCH] Look in Debian's multiarch libs too
Guido Günther
agx at sigxcpu.org
Thu Sep 13 21:06:00 UTC 2012
On Thu, Sep 13, 2012 at 10:46:30AM -0600, Eric Blake wrote:
> On 09/13/2012 09:36 AM, Guido Günther wrote:
> > On Thu, Sep 13, 2012 at 02:35:03PM +0200, =?UTF-8?q?Guido=20G=C3=BCnther?= wrote:
> >> I checked for libnetcf.so which is in /usr/lib/ but not for libnl1 which
> >> is already in a multiarch directory (`/usr/lib/i386-linux-gnu/libnl.so).
> >> So this fixes it:
> >>
>
> >> + for dir in /usr/lib64 /usr/lib /usr/lib/*/libnl.so; do
> >
> > In fact, what I send in is crap since I need to put a directory here not
> > the library itself.
>
> Yeah, I was about to point that out, too, until I saw your next email.
>
> At any rate, it looks like you are acking my patch, and I am now acking
> yours :)
Yes. ACK.
>
> > It turned out that a "git clean -dfx" fixed the
> > problem too - although I don't know why.
>
> I do - because either of our patches in isolation is sufficient to avoid
> aborting configure; but my patch in isolation still costs you the time
> to probe libnl-3. Both patches are useful.
>
> > Nevertheless the attched patch
> > would make things more robust since Debian is moving to multiarch [1]
> > and more and more libs move to /usr/lib/<multiarch-specifier> so we'd be
> > safe in the future. O.k. to apply the attached version?
>
> ACK.
Pushed. Thanks.
-- Guido
>
> --
> Eric Blake eblake at redhat.com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
More information about the libvir-list
mailing list