[Freeipa-devel] [PATCH] 0014

Florence Blanc-Renaud flo at redhat.com
Fri Sep 2 07:05:58 UTC 2016


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-frenaud-0014-2-Fix-ipa-certupdate-for-CA-less-installation.patch
Type: text/x-patch
Size: 1740 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160902/086aa22d/attachment.bin>


More information about the Freeipa-devel mailing list