#143 - Remove service loader META-INF services file for custom modules

Now that custom modules can have no default constructor they can not be loaded via ServiceLoader. This should not matter as it really didn't make sense to service load the custom modules anyway.

Note that the 'default' scope modules still are expected to be service loaded. That does mean that we do not create a specific constructor for 'default' scope modules. External dependencies for default modules must still be provided via withBean()

Author: rbygrave

inject-generator/src/main/java/io/avaje/inject/generator/AllScopes.java

@@ -5,9 +5,6 @@
 import javax.annotation.processing.RoundEnvironment;
 import javax.lang.model.element.Element;
 import javax.lang.model.element.TypeElement;
-import javax.tools.FileObject;
-import java.io.IOException;
-import java.io.Writer;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -50,39 +47,6 @@ void write(boolean processingOver) {
     for (Data value : scopeAnnotations.values()) {
       value.write(processingOver);
     }
-    if (processingOver) {
-      writeModuleCustomServicesFile();
-    }
-  }
-
-  private void writeModuleCustomServicesFile() {
-    if (scopeAnnotations.isEmpty()) {
-      return;
-    }
-    try {
-      FileObject jfo = context.createMetaInfModuleCustom();
-      if (jfo != null) {
-        Writer writer = jfo.openWriter();
-        for (Data value : scopeAnnotations.values()) {
-          final String moduleFullName = value.moduleFullName();
-          if (moduleFullName == null) {
-            // an empty module, custom scope with no beans
-            final TypeElement typeElement = value.annotationType();
-            if (typeElement != null) {
-              context.logWarn("Empty module for "+typeElement);
-            }
-          } else {
-            writer.write(moduleFullName);
-            writer.write("\n");
-          }
-        }
-        writer.close();
-      }
-
-    } catch (IOException e) {
-      e.printStackTrace();
-      context.logError("Failed to write services file " + e.getMessage());
-    }
   }
 
   void readModules(List<String> customScopeModules) {
@@ -125,13 +89,5 @@ static class Data {
     void write(boolean processingOver) {
       scopeInfo.write(processingOver);
     }
-
-    String moduleFullName() {
-      return scopeInfo.moduleFullName();
-    }
-
-    TypeElement annotationType() {
-      return scopeInfo.annotationType();
-    }
   }
 }