Issue1671
Created on 2010-11-01.03:50:02 by sergei175, last changed 2010-11-09.22:57:01 by otmarhumbel.
File name |
Uploaded |
Description |
Edit |
Remove |
FooTest.java
|
public.marvin,
2010-11-09.17:44:16
|
|
|
|
1671-patch.txt
|
otmarhumbel,
2010-11-09.22:54:45
|
the fix |
|
|
msg6214 (view) |
Author: Sergei (sergei175) |
Date: 2010-11-01.03:50:00 |
|
Jython interpreter complains:
> Bootstrapping class not in Py.BOOTSTRAP_TYPES
for applications incorporating jython interpreter, such as
http://linux.wareseeker.com/Programming/jyconsole-1.3.zip/329101
(replace old jython with new from 2.5.2r2)
this started to show up in jython 2.5.2b1 and still same
warning is in jython 2.5.2r2
Was tested using java 16.22 on ubuntu 10.10
|
msg6220 (view) |
Author: Oti Humbel (otmarhumbel) |
Date: 2010-11-02.20:17:48 |
|
Sorry, was not able to download the .jar, even after registering as a new artenum user.
If you mail it to me directly, I would try to reproduce this.
(ohumbel /a/t/ yahoo.com)
|
msg6237 (view) |
Author: marvin greenberg (public.marvin) |
Date: 2010-11-09.17:44:16 |
|
Here's a simple file that demonstrates the problem, just compile it and run it. I suspect that this will fix it,
--- src/org/python/core/Py.java.orig 2010-11-09 10:12:23.000000000 -0500
+++ src/org/python/core/Py.java 2010-11-09 08:33:15.000000000 -0500
@@ -94,6 +94,7 @@
BOOTSTRAP_TYPES.add(PyType.class);
BOOTSTRAP_TYPES.add(PyBuiltinCallable.class);
BOOTSTRAP_TYPES.add(PyDataDescr.class);
+ BOOTSTRAP_TYPES.add(PyStringMap.class);
}
/** A unique object to indicate no conversion is possible
but I couldn't get the existing jython build.xml to build a jar even after chasing down dependencies for an hour.
|
msg6239 (view) |
Author: Oti Humbel (otmarhumbel) |
Date: 2010-11-09.21:15:05 |
|
Marvin,
thanks for showing how simple it is to reproduce!
I am testing out the fix
|
msg6240 (view) |
Author: Oti Humbel (otmarhumbel) |
Date: 2010-11-09.22:21:21 |
|
Unfortunately the fix is not so simple.
PyStringMap is not bootstrapped in any case, so adding it to BOOTSTRAP_TYPES produces a symmetric warning.
My strategy is to lazily add it if necessary.
|
msg6241 (view) |
Author: Oti Humbel (otmarhumbel) |
Date: 2010-11-09.22:54:45 |
|
The workspace diff in 1671-patch.txt fixes the issue and passes all regrtests
|
msg6242 (view) |
Author: Oti Humbel (otmarhumbel) |
Date: 2010-11-09.22:57:01 |
|
fixed in revision 7170
|
|
Date |
User |
Action |
Args |
2010-11-09 22:57:01 | otmarhumbel | set | status: open -> closed resolution: fixed messages:
+ msg6242 |
2010-11-09 22:54:46 | otmarhumbel | set | files:
+ 1671-patch.txt messages:
+ msg6241 |
2010-11-09 22:21:21 | otmarhumbel | set | messages:
+ msg6240 |
2010-11-09 21:15:05 | otmarhumbel | set | assignee: otmarhumbel messages:
+ msg6239 |
2010-11-09 17:44:17 | public.marvin | set | files:
+ FooTest.java nosy:
+ public.marvin messages:
+ msg6237 |
2010-11-02 20:17:49 | otmarhumbel | set | nosy:
+ otmarhumbel messages:
+ msg6220 |
2010-11-01 03:50:02 | sergei175 | create | |
|