issue #190: Module classpath should filter invalid items

Author: siordache

src/main/java/org/javamodularity/moduleplugin/tasks/CompileJavaTaskMutator.java

@@ -11,8 +11,10 @@
 import org.javamodularity.moduleplugin.extensions.PatchModuleContainer;
 import org.javamodularity.moduleplugin.internal.MutatorHelper;
 
+import java.io.File;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.stream.Collectors;
 
 class CompileJavaTaskMutator {
     private static final Logger LOGGER = Logging.getLogger(CompileJavaTaskMutator.class);
@@ -62,10 +64,15 @@ private List<String> buildCompilerArgs(JavaCompile javaCompile) {
                 .map(AbstractCompile::getDestinationDir)
                 .forEach(dir -> patchModuleContainer.addDir(moduleName, dir.getAbsolutePath()));
 
+        // Keep only valid module-path entries (https://github.com/java9-modularity/gradle-modules-plugin/issues/190)
+        FileCollection filteredClasspath = project.files(compileJavaClasspath.getFiles().stream()
+                .filter(f -> f.getName().endsWith(".jar") || f.getName().endsWith(".jmod"))
+                .collect(Collectors.toList()).toArray());
+
         var compilerArgs = new ArrayList<>(javaCompile.getOptions().getCompilerArgs());
-        patchModuleContainer.buildModulePathOption(compileJavaClasspath)
+        patchModuleContainer.buildModulePathOption(filteredClasspath)
                 .ifPresent(option -> option.mutateArgs(compilerArgs));
-        patchModuleContainer.mutator(compileJavaClasspath).mutateArgs(compilerArgs);
+        patchModuleContainer.mutator(filteredClasspath).mutateArgs(compilerArgs);
 
         moduleOptions.mutateArgs(compilerArgs);
         MutatorHelper.configureModuleVersion(helper(), compilerArgs);

src/main/java/org/javamodularity/moduleplugin/tasks/CompileTask.java

@@ -37,15 +37,6 @@ public void configureCompileJava() {
     }
 
     private void configureCompileJava(JavaCompile compileJava) {
-        project.getConfigurations().stream()
-                .flatMap(configuration -> configuration.getDependencies().stream())
-                .filter(dependency -> dependency instanceof ProjectDependency)
-                .map(dependency -> ((ProjectDependency) dependency).getDependencyProject().getTasks())
-                .map(tasks -> tasks.findByName(CompileModuleOptions.COMPILE_MODULE_INFO_TASK_NAME))
-                .filter(Objects::nonNull);
-
-
-
         var moduleOptions = compileJava.getExtensions().create("moduleOptions", CompileModuleOptions.class, project);
         project.afterEvaluate(p -> {
             adjustMainClass(compileJava);