[bugfix] Store the path of Jar files in each EXPath Package's .exist/classpath.txt file relative to the package's content/ sub-folder.

Closes #4901

Author: adamretter

exist-core/src/main/java/org/exist/repo/ClasspathHelper.java

@@ -170,10 +170,17 @@ private static void scanPackageDir(Package pkg, Classpath classpath, Path module
                 try (final BufferedReader reader = Files.newBufferedReader(cp)) {
                     String line;
                     while ((line = reader.readLine()) != null) {
-                        if (!Files.exists(Paths.get(line))) {
-                            LOG.warn("Unable to add '" + line + "' to the classpath for EXPath package: " + pkg.getName() + ", as the file does not exist!");
+                        Path p = Paths.get(line);
+                        if (!p.isAbsolute()) {
+                            final FileSystemStorage.FileSystemResolver res = (FileSystemStorage.FileSystemResolver) pkg.getResolver();
+                            p = res.resolveComponentAsFile(line);
+                        }
+                        p = p.normalize().toAbsolutePath();
+
+                        if (!Files.exists(p)) {
+                            LOG.warn("Unable to add '" + p + "' to the classpath for EXPath package: " + pkg.getName() + ", as the file does not exist!");
                         } else {
-                            classpath.addComponent(line);
+                            classpath.addComponent(p.toString());
                         }
                     }
                 }

exist-core/src/main/java/org/exist/repo/ExistPkgExtension.java

@@ -32,15 +32,12 @@
 import javax.xml.stream.XMLStreamConstants;
 import javax.xml.stream.XMLStreamException;
 import javax.xml.stream.XMLStreamReader;
-import javax.xml.transform.stream.StreamSource;
 import java.io.IOException;
 import java.io.Writer;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.nio.file.Paths;
-import java.util.Set;
 
 /**
  * Handle the exist.xml descriptor in an EXPath package.
@@ -158,23 +155,22 @@ private void setupPackage(Package pkg, ExistPkgInfo info)
                 throw new PackageException("Impossible to create directory: " + exist);
             }
         }
-        final Set<String> jars = info.getJars();
+
         try(final Writer out = Files.newBufferedWriter(classpath)) {
-            for (final String jar : jars) {
-                StreamSource jar_src;
+            for (final String jar : info.getJars()) {
+
                 try {
-                    jar_src = res.resolveComponent(jar);
+                    res.resolveComponent(jar);
                 } catch (final NotExistException ex) {
-                    final String msg = "Inconsistent package descriptor, the JAR file is not in the package: ";
+                    final String msg = "Inconsistent package descriptor, the JAR file is not in the EXPath package: ";
                     throw new PackageException(msg + jar, ex);
                 }
-                final URI uri = URI.create(jar_src.getSystemId());
-                final Path file = Paths.get(uri);
-                out.write(file.normalize().toString());
-                out.write("\n");
+
+                out.write(jar);
+                out.write('\n');
             }
         } catch (final IOException ex) {
-            throw new PackageException("Error writing the eXist classpath file: " + classpath, ex);
+            throw new PackageException("Error writing the eXist classpath file '" + classpath + "' for the EXPath package: " + pkg.getName(), ex);
         }
     }