[Freeipa-devel] [PATCH 0118] Allow to disable service (in LDAP)

[Jan Cholasta]

Hi,

Dne 1.9.2014 v 16:57 Martin Basti napsal(a):
> This patch allows to disable service in LDAP to prevents service to be
> started by "ipactl restart"
>
> Required by DNSSEC
>
> Patch attached

I don't think the extra argument in ldap_enable is necessary. It should 
enable the service no matter if the entry existed before or not.

Similarly, in ldap_disable you should not raise an error when the entry 
is not found, because that already makes the service disabled.

Honza

-- 
Jan Cholasta