Improved @filename support

src/main/java/rife/bld/operations/JlinkOperation.java

@@ -100,7 +100,7 @@ public class JlinkOperation extends AbstractToolProviderOperation<JlinkOperation
         return options_;
     }
 
-    // Shouldn't be needed, but for some reason jlink doesn't like @filename when called via ToolProvider
+    // Shouldn't be needed, but jlink doesn't support @filename when called via ToolProvider
     private List<String> parseOptions() throws FileNotFoundException {
         var list = new ArrayList<String>();
 
@@ -109,10 +109,10 @@ public class JlinkOperation extends AbstractToolProviderOperation<JlinkOperation
                 while (scanner.hasNext()) {
                     var splitLine = scanner.nextLine().split("--");
                     for (String args : splitLine) {
-                        if (!args.isEmpty()) {
+                        if (!args.isBlank()) {
                             var splitArgs = args.split(" ", 2);
                             list.add("--" + splitArgs[0]);
-                            if (splitArgs.length > 1 && !splitArgs[1].isEmpty()) {
+                            if (splitArgs.length > 1 && !splitArgs[1].isBlank()) {
                                 list.add(splitArgs[1]);
                             }
                         }

src/test/java/rife/bld/operations/TestJlinkOperation.java

@@ -110,7 +110,14 @@ public class TestJlinkOperation {
 
     @Test
     void testOptions() {
-        var jlink = new JlinkOperation().options("src/test/resources/jlink/options_verbose.txt");
+        var jlink = new JlinkOperation().options("src/test/resources/jlink/options_verbose.txt",
+                "src/test/resources/jlink/options_version.txt");
+        assertDoesNotThrow(jlink::execute);
+    }
+
+    @Test
+    void testParseOptions() {
+        var jlink = new JlinkOperation().options("src/test/resources/jlink/options_jlink.txt");
         assertDoesNotThrow(jlink::execute);
     }
 

src/test/java/rife/bld/operations/TestJmodOperation.java

@@ -142,7 +142,8 @@ public class TestJmodOperation {
 
     @Test
     void testOptions() {
-        var jpackage = new JpackageOperation().options("src/test/resources/jlink/options_version.txt");
+        var jpackage = new JpackageOperation().options("src/test/resources/jlink/options_verbose.txt",
+                "src/test/resources/jlink/options_version.txt");
         assertDoesNotThrow(jpackage::execute);
     }
 

src/test/java/rife/bld/operations/TestJpackageOperation.java

@@ -199,7 +199,8 @@ public class TestJpackageOperation {
 
     @Test
     void testOptions() {
-        var jpackage = new JpackageOperation().options("src/test/resources/jlink/options_verbose.txt");
+        var jpackage = new JpackageOperation().options("src/test/resources/jlink/options_verbose.txt",
+                "src/test/resources/jlink/options_version.txt");
         assertDoesNotThrow(jpackage::execute);
     }
 

src/test/resources/jlink/options_jlink.txt

@@ -0,0 +1,2 @@
+ --verbose  --version
+    --list-plugins

src/test/resources/jlink/options_verbose.txt

@@ -1 +1 @@
---verbose --version
+--verbose