[ofa-general] Re: [PATCH 0/3] ibnetdiscover library "libibnetdisc"

[Sasha Khapyorsky]

Hi Ira,

On 13:49 Mon 24 Nov     , Ira Weiny wrote:
> 
> As long as the library exists any dependant package can of course use the
> library from whatever package we chose (libibnetdisc or infiniband-diags).  We
> have some code which is prototyped against ibnetdiscover but we plan on using
> this library instead.  This would be separate from infiniband-diags.  But we
> can just as easily put a dependancy on infiniband-diags as on libibnetdisc.

Yes, it is possible to make dependency. But I'm getting complains about
too many in-management dependencies even now.

> The fact is that it was actually easier to put this in a new package rather
> than try and integrate with infiniband-diags.

I would disagree. In later case we need to deal with only one logical
change and not with all "new package" issues.

> I thought it made for a very
> clean conversion by putting the library in as a new patch and then we could
> convert the diags as appropriate.
> 
> Anyway, I will integrate it as you say and resubmit the patch.

Thanks.

Sasha