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

[Martin Basti]

On 02/09/14 09:10, Jan Cholasta wrote:
> 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
>
Updated patch attached

-- 
Martin Basti

-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0118.2-LDAP-disable-service.patch
Type: text/x-patch
Size: 3219 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140902/b97719aa/attachment.bin>