Re: [PATCH] rdma-core: Add missing systemd-devel dependency to the SUSE rdma-core.spec

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

 




Le 07/12/2017 à 15:59, Jason Gunthorpe a écrit :
> On Thu, Dec 07, 2017 at 10:57:08AM +0100, Nicolas Morey-Chaisemartin wrote:
>
>> On a recent systemd, systemd-devel provides all those (probably for compat reason during migration):
>> $ rpm -q --provides systemd-devel
>> pkgconfig(libsystemd) = 228
>> pkgconfig(libsystemd-daemon) = 228
>> pkgconfig(libsystemd-id128) = 228
>> pkgconfig(libsystemd-journal) = 228
>> pkgconfig(libsystemd-login) = 228
>>
>>
>> So I can porobably safely changes the BuildRequires to the libsystemd-XXXX.
>> If I'm not mistaken, we just need the -daemon and -journal ones.
> cmake checks for all id128 as well when it runs in compat mode, so you
> need all three to make it happy.
>
> I wouldn't change the main spec file, maybe protect the 2nd option
> with a ifdef for old suse versions?

I don't really mind either way.

However is there any chance that it will work ?
I'm pretty sure there are some (if not many) ABI changes from ibverbs 1.2.0 to rdma-core v16.
This means that all dependent packages needs to be rebuilt, and some probably updated too.

>From my SUSE package maintainer perspective, that is something that should not be done and SUSE will not provide any official support for that.
Regarding upstream we can't really forbid people to do this, just try to point them in a "sane" direction :)

Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux