On 02.02.2016 12:18, Jan Cholasta wrote:
On 2.2.2016 11:46, Jan Cholasta wrote:
On 2.2.2016 11:41, Martin Babinsky wrote:
On 02/02/2016 09:33 AM, Jan Cholasta wrote:
On 1.2.2016 14:54, Martin Basti wrote:
On 01.02.2016 13:55, Martin Babinsky wrote:
LGTM, works for me, tested on both ca-less server and CA-full server.
Because patch is touching certmonger I would like to get final ACK
This is suspicious:
- if is_ca_enabled():
Why is it necessary?
I don't know, you tell me:
What I meant is why is the change necessary, not why is the original
Ah, the original code didn't have the condition. LGTM then.
LGTM + LGTM = ACK :)
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code