Issue1607
Created on 2010-05-06.15:35:39 by sven, last changed 2010-08-06.06:14:41 by zyasoft.
msg5762 (view) |
Author: Sven Klemm (sven) |
Date: 2010-05-06.15:35:37 |
|
In Jython the __import__ function works differently than the one in CPython, CPython allows an empty string as globals while Jython doesn't.
__import__('csv','') works in CPython while it doesn't work in Jython
svekle@blackdevil:~> jython
Jython 2.5.1+ (unknown:exported, May 6 2010, 12:31:38)
[Java HotSpot(TM) Client VM (Sun Microsystems Inc.)] on java1.6.0_15
Type "help", "copyright", "credits" or "license" for more information.
>>> __import__('csv','')
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: str indices must be integers
svekle@blackdevil:~> python2.5
Python 2.5.4 (r254:67916, Jan 20 2010, 21:44:03)
[GCC 4.4.1] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> __import__('csv','')
<module 'csv' from '/usr/lib/python2.5/csv.pyc'>
|
msg5794 (view) |
Author: Philip Jenvey (pjenvey) |
Date: 2010-06-03.02:37:07 |
|
I seem to recall this issue coming up before, but I'm not sure and can't recall the details.
Regardless, I strongly recommend you use None here instead. '' here is bad practice, and could go away in the future
|
msg5944 (view) |
Author: Jim Baker (zyasoft) |
Date: 2010-08-06.05:54:03 |
|
Please use a mapping for globals or locals. CPython behavior here is too sloppy to be worthwhile replicating for Jython.
|
msg5945 (view) |
Author: Jim Baker (zyasoft) |
Date: 2010-08-06.06:14:41 |
|
I should point out, if the purpose of this is to workaround some code that needs the CPython behavior, you can always patch the builtin __import__ function. Some examples of changing builtins are in test_sys_jy.py, and could be introduced through possibly sitecustomize.py.
|
|
Date |
User |
Action |
Args |
2010-08-06 06:14:41 | zyasoft | set | messages:
+ msg5945 |
2010-08-06 05:54:03 | zyasoft | set | status: open -> closed resolution: wont fix messages:
+ msg5944 nosy:
+ zyasoft |
2010-06-17 07:49:05 | thijs | set | nosy:
+ thijs |
2010-06-03 02:37:07 | pjenvey | set | priority: low nosy:
+ pjenvey messages:
+ msg5794 |
2010-05-06 15:35:39 | sven | create | |
|