[Freeipa-devel] [PATCH] 0014

Tomas Krizek tkrizek at redhat.com
Mon Sep 5 08:42:08 UTC 2016


On 09/02/2016 09:05 AM, Florence Blanc-Renaud wrote:
> On 09/02/2016 08:08 AM, Jan Cholasta wrote:
>> On 1.9.2016 19:37, Tomas Krizek wrote:
>>> On 09/01/2016 03:58 PM, Florence Blanc-Renaud wrote:
>>>> Hi,
>>>>
>>>> please find attached a patch for ipa-certupdate in CA-less deployment.
>>>> https://fedorahosted.org/freeipa/ticket/6288
>>>>
>>>> Flo.
>>>>
>>>>
>>>>
>>> The patch is malformed, but you can simply delete the very first
>>> character to fix it.
>>>
>>> Other than that, patch works as expected -> ACK.
>>
>> Nitpick: please avoid C-isms such as "if (ca_enabled):".
>>
> Hi all,
>
> thanks for the review. Please find an updated patch version. Quite 
> difficult to get rid of typing habits...
>
> Flo
>
ACK

-- 
Tomas Krizek




More information about the Freeipa-devel mailing list