circular imports make python3 unhappy about iniparse

[GoogleCodeExporter]

Original comment by tim.lauridsen on 5 May 2010 at 9:37

Attachments:

• python3.diff

[GoogleCodeExporter]

I have added a diff file to see the changes made to the code by the 2to3 tool.

Original comment by tim.lauridsen on 5 May 2010 at 9:38

[GoogleCodeExporter]

Yeah - create a iniparse.utils module and move tidy to it.  Also, change the
corresponding lines in iniparse/__init__.py

Original comment by [email protected] on 7 May 2010 at 2:21

[GoogleCodeExporter]

Based on a 1-minute look at the diff, the 2to3 changes look mostly sane, except 
for a
few extra list creations that I think could be avoided (in the
iterkeys/itervalues/iteritems functions).

I think these changes are sufficient if we want iniparse to work with files 
opened in
text mode.  However, currently iniparse supports both byte and unicode strings. 
 If
we want to preserve that functionality, we'll probably have to make more 
changes.

Original comment by [email protected] on 7 May 2010 at 2:31

[GoogleCodeExporter]

ok, I have commited the changes for moving tidy

http://code.google.com/p/iniparse/source/detail?r=139

I was not my plan to add the 2to3 diff to upstream at this point in time, i will
modify the rpm build spec file to build both a python 2.x version and python 
3.x one
by running 2to3 on the sources before the build. So no need to convert upstream
sources at this point in time.

Original comment by tim.lauridsen on 8 May 2010 at 7:10

[GoogleCodeExporter]

sounds good.

Original comment by [email protected] on 8 May 2010 at 7:09

• Changed state: Fixed