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

[Martin Basti]

On 02/09/14 11:59, Martin Basti wrote:
> 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
>
>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
Please review, this should be in 4.1

-- 
Martin Basti

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140919/48a7f8d4/attachment.htm>