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

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to