On Wed, 2015-03-11 at 12:42 +0100, Petr Spacek wrote:
> I would like to see new code compatible with Python 3. Here I'm not
> sure what is the generic solution for xrange but in this particular
> case I would recommend you to use just range. Attempts variable should
> have small values so the x/range differences do not matter here.

To be honest, I do not think we should care for this code specifically.
The move to Python3 will require python-krbV code to be completely
removed, and we should use python-gssapi instead. I will help with the
"how do I kinit" part once we can do that.

Now, I do not know if it is worth NACKing this patch in that sense, or
letting it through, it depends on whether we want to be able to backport
this specific fix or whether it is ok to have it only available on
python-gssapi capable machines.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

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