[SSSD] [PATCH] Build cifs plugin depending on the existence of cifsidmap.h

Jakub Hrozek jhrozek at redhat.com
Mon Nov 4 09:51:27 UTC 2013


On Fri, Nov 01, 2013 at 06:41:20PM +0100, Sumit Bose wrote:
> Hi,
> 
> with this patch it is not required anymore to have cifs-utils-devel
> installed for the default build.
> 
> Fixes https://fedorahosted.org/sssd/ticket/2125
> 
> bye,
> Sumit

Hi,

I think it would also be nice to add explicit
"with_cifs_utils_plugin_option --enable-cifs-idmap-plugin" to
contrib/sssd.spec.in where available. That way, even if cifs-utils-devel
was renamed configure would error out (assuming we want the CIFS support
enabled in upstream RPMs by defaul, which I think we do and Simo's point
was mostly about building from source).

The configure option itself works nicely, I tested with and without
cifs-utils-devel, explicitly disabling or enabling the support.



More information about the sssd-devel mailing list