On 5.9.2016 10:42, Tomas Krizek wrote:


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

Pushed to:
master: b36ee723b77a2721f4200d5df02268a9bd6a60b5
ipa-4-4: 1b8f6ec58600ad4bbfb538ddcff659ea1ba2c324

--
Jan Cholasta

--
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