Issue1373

Resource files in jars that are added via sys.path won't be found by
getRessource. Consider the pck.jar file that contains pck/Main.java and
the resource pck/Main.txt:

    >>> import sys
    >>> sys.path.append("pck.jar")
    >>> import pck
    *sys-package-mgr*: processing new jar,
'/home/boisgera/SANDBOX/JYTHON-BUGS/RESOURCE/pck.jar'
    >>> pck.Main.getResource("Main.txt") is None
    True
    >>> pck.Main.getClassLoader()
    org.python.core.SyspathJavaLoader@13d21d6

On the other hand, if 'pck.jar' is already in the CLASSPATH, I end up with:

    >>> import pck
    >>> pck.Main.getResource("Main.txt")
   
jar:file:/home/boisgera/SANDBOX/JYTHON-BUGS/RESOURCE/pck.jar!/pck/Main.txt
    >>> pck.Main.getClassLoader()
    sun.misc.Launcher$AppClassLoader@17182c1

I guess that 'findResource' should be implemented in SyspathJavaLoader
for this to work. The method 'getResourceAsStream' being already
implemented, I guess that most of the work has already been done ... right ?

Yea it looks like all the work is done it just needs a reshuffling of 
implemented methods

Included in the patch is a test case which requires a jar identical to the 
pck.jar file attached to this issue, but compiled under java 5 and renamed 
to bug1373.jar. I ran regrtest with the patch applied and saw no failures.

This problem persists also with the new SyspathJavaLoader but the attached patch is obsolete. 

I've written a new patch:

Index: src/org/python/core/SyspathJavaLoader.java
===================================================================
--- src/org/python/core/SyspathJavaLoader.java	(revision 6980)
+++ src/org/python/core/SyspathJavaLoader.java	(working copy)
@@ -134,7 +134,7 @@
                 ZipEntry ze = archive.getEntry(entryRes);
                 if (ze != null) {
                 	try {
-						return new URL("jar:" + entry.__str__().toString() + "!/" + entryRes);
+						return new URL("jar:file:" + entry.__str__().toString() + "!/" + entryRes);
 					} catch (MalformedURLException e) {
 						throw new RuntimeException(e);
 					}
@@ -143,7 +143,11 @@
             }
             String dir = sys.getPath(entry.__str__().toString());
             try {
-				return new File(dir, res).toURI().toURL();
+				File resource = new File(dir, res);
+				if (!resource.exists()) {
+					continue;
+				}
+				return resource.toURI().toURL();
 			} catch (MalformedURLException e) {
 				throw new RuntimeException(e);
 			}


The unit test may be "recycled"

I seem to have encountered this bug as well. This may be the issue I've brought up a couple of times in the past (going back to 2.1) being unable to have sys.path-added modules load their own resources.

In the past, I've used jclasspath (http://www.fiber-space.de/jynx/doc/jclasspath.html) to work around these issues (by appending the necessary classpath to the system Classpath, which requires bypassing java attribute accessibility rules).

I'm eager to see if this patch corrects these issues. Would it be possible to compile a .class file so I can inject it in my jython.jar and test, and if so would someone be willing to do so?

Try with this jar. It contains the compiled patch.

I meant jython-dev.jar, that I've just attached.

I'm trying to test the compiled patch, but I'm not sure how. When I copy jython-dev.jar to my Jython home directory, I get a traceback (below). If I just inject the SyspathJavaLoader.class into the stock 2.5.1 jython.jar, I get only an 'Exception in thread "main"' error (and the program exits).

Any suggestions?

Exception in thread "main" java.lang.NoClassDefFoundError: org/jruby/ext/posix/POSIXHandler
        at org.python.modules.Setup.<clinit>(Setup.java:62)
        at org.python.core.PySystemState.initBuiltins(PySystemState.java:1000)
        at org.python.core.PySystemState.doInitialize(PySystemState.java:837)
        at org.python.core.PySystemState.initialize(PySystemState.java:756)
        at org.python.core.PySystemState.initialize(PySystemState.java:706)
        at org.python.core.PySystemState.initialize(PySystemState.java:699)
        at org.python.util.jython.run(jython.java:150)
        at org.python.util.jython.main(jython.java:129)
Caused by: java.lang.ClassNotFoundException: org.jruby.ext.posix.POSIXHandler
        at java.net.URLClassLoader$1.run(URLClassLoader.java:200)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.net.URLClassLoader.findClass(URLClassLoader.java:188)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:303)
        at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:301)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:248)
        at java.lang.ClassLoader.loadClassInternal(ClassLoader.java:316)
        ... 8 more

Fixed by r7080

Thanks to everyone for getting this patch applied. Is there someplace I can pick up a nightly build so I can test this against our failing case?

Confirmed fixed in 2.5.2b2. Thanks to everyone for making this happen.