[Bug 169744] Review Request: libmthca - Mellanox hardware support for libibverbs

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmthca - Mellanox hardware support for libibverbs


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=169744





------- Additional Comments From rolandd at cisco.com  2005-11-03 16:23 EST -------
> - The devel package includes a static library but there are no header 
>     files -- I assume thats because this is a "plug-in library" for 
>     libibverbs and it uses the libibverbs-devel headers so it doesn't 
>     actually have to provide any headers itself, right?  If so, thats 
>     fine but then you should probably have libmthca-devel Require: 
>     the libibverbs-devel package

Right, there are no header files because nothing except libibverbs
calls functions in libibverbs.  The static library is just there because
some people have found it useful to link everything staticly into their
app (libibverbs finds static libmthca via dlopen(NULL, )).

You're probably right that libmthca-devel needs to Require: libibverbs-devel,
since libmthca does call libibverbs functions.

> - a shared library is installed but the usual post/postun ldconfig 
>     scripts are not run -- is that really OK?

Do you mean /usr/lib/infiniband/mthca.so?  Again, that's a "plug-in"
that's dlopen()ed by libibverbs, so nothing directly links to it.  So
I don't think we need to do any ldconfig stuff, right?


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.