commit 29bea3409ced14a269d07cf1d91d60aae872d49e
author Terence Yim <terencey@google.com> Sun Mar 15 15:07:16 2020 -0700
committer Terence Yim <terencey@google.com> Sun Mar 15 18:37:47 2020 -0700
tree a251b9f43cf6ad992ee31f76ab38cae61a6e31ee
parent 71f57be7682087dc0e6b3ad77c9d0159a3ea8782

(TWILL-259) Expands environment variables in classpath

This fixes #89 on GitHub.

Signed-off-by: Terence Yim <terencey@google.com>

twill-core/src/main/java/org/apache/twill/launcher/TwillLauncher.java

diff --git a/twill-core/src/main/java/org/apache/twill/launcher/TwillLauncher.java b/twill-core/src/main/java/org/apache/twill/launcher/TwillLauncher.java
index 056639f..3e41592 100644
--- a/twill-core/src/main/java/org/apache/twill/launcher/TwillLauncher.java
+++ b/twill-core/src/main/java/org/apache/twill/launcher/TwillLauncher.java
@@ -85,14 +85,8 @@
 
     // For backward compatibility, sort jars from twill and jars from application together
     // With TWILL-179, this will change as the user can have control on how it should be.
-    List<File> libJarFiles = listJarFiles(new File(appJarDir, "lib"), new ArrayList<File>());
-    Collections.sort(listJarFiles(new File(twillJarDir, "lib"), libJarFiles), new Comparator<File>() {
-      @Override
-      public int compare(File file1, File file2) {
-        // order by the file name only. If the name are the same, the one in application jar will prevail.
-        return file1.getName().compareTo(file2.getName());
-      }
-    });
+    List<File> libJarFiles = listJarFiles(new File(appJarDir, "lib"), new ArrayList<>());
+    listJarFiles(new File(twillJarDir, "lib"), libJarFiles).sort(Comparator.comparing(File::getName));
 
     // Add the app jar, resources jar and twill jar directories to the classpath as well
     for (File dir : Arrays.asList(appJarDir, twillJarDir)) {
@@ -115,7 +109,7 @@
     }
 
     addClassPathsToList(urls, new File(Constants.Files.RUNTIME_CONFIG_JAR, Constants.Files.APPLICATION_CLASSPATH));
-    return urls.toArray(new URL[urls.size()]);
+    return urls.toArray(new URL[0]);
   }
 
   /**
@@ -148,7 +142,7 @@
     try (BufferedReader reader = Files.newBufferedReader(classpathFile.toPath(), StandardCharsets.UTF_8)) {
       String line = reader.readLine();
       if (line != null) {
-        for (String path : line.split(":")) {
+        for (String path : expand(line).split(":")) {
           urls.addAll(getClassPaths(path.trim()));
         }
       }
@@ -160,7 +154,7 @@
     if (classpath.endsWith("/*")) {
       // Grab all .jar files
       File dir = new File(classpath.substring(0, classpath.length() - 2));
-      List<File> files = listJarFiles(dir, new ArrayList<File>());
+      List<File> files = listJarFiles(dir, new ArrayList<>());
       Collections.sort(files);
       if (files.isEmpty()) {
         return singleItem(dir.toURI().toURL());