[ovs-dev] [PATCH] rhel: Add libcap-ng dependency.

Russell Bryant rbryant at redhat.com
Thu Oct 1 20:00:09 UTC 2015


On 10/01/2015 03:54 PM, Ben Pfaff wrote:
> On Thu, Oct 01, 2015 at 03:50:35PM -0400, Russell Bryant wrote:
>> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
>> the libcap-ng library.  It's packaged in Fedora, so go ahead and added
>> it to the Fedora spec file.
>>
>> Our default systemd unit files don't make use of the --user option that
>> requires this library, but conceivably someone may want to customize
>> them and use this option.
>>
>> Signed-off-by: Russell Bryant <rbryant at redhat.com>
> 
> I'd consider also adding --with-libcapng=yes to the configure command
> line.  That way, a build without libcap-ng installed will fail.  That
> will both root out problems in whatever RPM autobuilders happen to be
> running against OVS and make sure that every RPM build supports
> libcap-ng.
> 

The rpm build would fail anyway if it's not installed at all.  I tested
that.

It would not catch possible failures due to distro or package version
differences.  For example, if the header file was installed was
installed in a different location and the configure script didn't find
it, the package would still happily build now, but without libcap-ng
support.  --with-libcapng=yes would catch that, at least.

Thanks for the suggestion.  I'll send out a v2.

-- 
Russell Bryant



More information about the dev mailing list