[Freeipa-devel] [PATCHES 481-486] Metaclass and str modernization

Petr Viktorin pviktori at redhat.com
Thu Sep 3 17:23:10 UTC 2015


On 09/01/2015 04:47 PM, Jan Cholasta wrote:
> Hi,
> 
> the attached patches add some more modernization to our code.
> 
> Honza

481: ACK
482: ACK
483: ACK

You can push these without waiting on the later ones.


484:
To avoid merge conflicts later, perhaps it would be better to have

    if six.PY3:
        unicode = str

at the start of each affected file, instead of scattering changes in the
files?
(I can prepare the patch if you agree)


485:
six.binary_type is named "bytes" since Python 2.6. I think it would be
better to use that, to avoid another change when py2 is dropped.
(I can prepare the patch here, too)


486: ACK

-- 
Petr Viktorin




More information about the Freeipa-devel mailing list