On 26.8.2015 12:13, Martin Basti wrote:


On 08/26/2015 11:19 AM, Michael Šimáček wrote:


On 2015-08-26 10:43, Jan Cholasta wrote:
On 26.8.2015 10:37, Martin Basti wrote:
Master branch is broken, this patch fixes it.

Patch attached.

This is not a pylint error, but an actual bug.


Yes, commit e46d9236d19f714b67fdf2865f19146c3016f46d (yesterday
evening) added another reference to krbV after my patch for migration
from krbV was reviewed. This patch looks ok to me, I'd just change the
commit message (to "Remove leftover krbV reference" or something like
that).

Thanks,
Michael

Fixed.

updated patch attached.

ACK.

Pushed to master: 14a87632e5df30dd2b44a1ca49e6b308d2249c70

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