On 06/16/2016 01:38 PM, Petr Spacek wrote: > On 16.6.2016 12:09, Pavel Vomacka wrote: >> >> >> On 06/16/2016 12:06 PM, Pavel Vomacka wrote: >>> Hello, >>> >>> please review attached patch. >>> >>> https://fedorahosted.org/freeipa/ticket/5905 >>> >> Fixed commit message > > > LGTM from user's perspective but I did not review the code. >
Code looks OK, applies well, but I did not test yet. So if Petr's LGTM is functional test ACK, then ACK from me. -- Petr Vobornik -- 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