Merge "Better logging for codegen's generateModuleInfos"
[yangtools.git] / code-generator / maven-sal-api-gen-plugin / src / main / java / org / opendaylight / yangtools / maven / sal / api / gen / plugin / CodeGeneratorImpl.java
index 9320e5af202ddc7e2433a8f753e3d65525527841..a2b60e89d3454151f4388e8a2c3d73f9893d22f6 100644 (file)
@@ -7,18 +7,10 @@
  */
 package org.opendaylight.yangtools.maven.sal.api.gen.plugin;
 
-import java.io.BufferedWriter;
-import java.io.File;
-import java.io.IOException;
-import java.io.OutputStream;
-import java.io.OutputStreamWriter;
-import java.io.Writer;
-import java.util.Collection;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-
+import com.google.common.base.Joiner;
+import com.google.common.base.Preconditions;
+import com.google.common.collect.ImmutableSet;
+import com.google.common.collect.ImmutableSet.Builder;
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.project.MavenProject;
 import org.opendaylight.yangtools.binding.generator.util.BindingGeneratorUtil;
@@ -27,6 +19,8 @@ import org.opendaylight.yangtools.sal.binding.generator.impl.BindingGeneratorImp
 import org.opendaylight.yangtools.sal.binding.model.api.Type;
 import org.opendaylight.yangtools.sal.java.api.generator.GeneratorJavaFile;
 import org.opendaylight.yangtools.sal.java.api.generator.YangModuleInfoTemplate;
+import org.opendaylight.yangtools.yang.binding.BindingMapping;
+import org.opendaylight.yangtools.yang.binding.YangModelBindingProvider;
 import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 import org.opendaylight.yangtools.yang2sources.spi.BuildContextAware;
@@ -35,7 +29,16 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.sonatype.plexus.build.incremental.BuildContext;
 
-import com.google.common.base.Preconditions;
+import java.io.BufferedWriter;
+import java.io.File;
+import java.io.IOException;
+import java.io.OutputStream;
+import java.io.OutputStreamWriter;
+import java.io.Writer;
+import java.util.Collection;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
 
 public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware {
     private static final String FS = File.separator;
@@ -45,6 +48,7 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
 
     private static final Logger logger = LoggerFactory.getLogger(CodeGeneratorImpl.class);
     private MavenProject mavenProject;
+    private File resourceBaseDir;
 
     @Override
     public Collection<File> generateSources(final SchemaContext context, final File outputDir,
@@ -55,7 +59,7 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
 
         final BindingGenerator bindingGenerator = new BindingGeneratorImpl();
         final List<Type> types = bindingGenerator.generateTypes(context, yangModules);
-        final GeneratorJavaFile generator = new GeneratorJavaFile(buildContext, new HashSet<>(types));
+        final GeneratorJavaFile generator = new GeneratorJavaFile(buildContext, types);
 
         File persistentSourcesDir = null;
         if (additionalConfig != null) {
@@ -69,11 +73,38 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
         }
 
         List<File> result = generator.generateToFile(outputBaseDir, persistentSourcesDir);
+
+        result.addAll(generateModuleInfos(outputBaseDir, yangModules, context));
+        return result;
+    }
+
+    private Collection<? extends File> generateModuleInfos(File outputBaseDir, Set<Module> yangModules,
+                                                           SchemaContext context) {
+        Builder<File> result = ImmutableSet.builder();
+        Builder<String> bindingProviders = ImmutableSet.builder();
         for (Module module : yangModules) {
-            // TODO: add YangModuleInfo class
-            result.add(generateYangModuleInfo(outputBaseDir, module, context));
+            Builder<String> currentProvidersBuilder = ImmutableSet.builder();
+            // TODO: do not mutate parameters, output of a method is defined by its return value
+            Set<File> moduleInfoProviders = generateYangModuleInfo(outputBaseDir, module, context, currentProvidersBuilder);
+            ImmutableSet<String> currentProviders = currentProvidersBuilder.build();
+            logger.info("Adding ModuleInfo providers {}", currentProviders);
+            bindingProviders.addAll(currentProviders);
+            result.addAll(moduleInfoProviders);
         }
-        return result;
+
+        result.add(writeMetaInfServices(resourceBaseDir, YangModelBindingProvider.class, bindingProviders.build()));
+        return result.build();
+    }
+
+    private File writeMetaInfServices(File outputBaseDir, Class<YangModelBindingProvider> serviceClass,
+            ImmutableSet<String> services) {
+        File metainfServicesFolder = new File(outputBaseDir, "META-INF" + File.separator + "services");
+        metainfServicesFolder.mkdirs();
+        File serviceFile = new File(metainfServicesFolder, serviceClass.getName());
+
+        String src = Joiner.on('\n').join(services);
+
+        return writeFile(serviceFile, src);
     }
 
     public static final String DEFAULT_OUTPUT_BASE_DIR_PATH = "target" + File.separator + "generated-sources"
@@ -93,7 +124,8 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
     }
 
     @Override
-    public void setLog(Log log) {}
+    public void setLog(Log log) {
+    }
 
     @Override
     public void setAdditionalConfig(Map<String, String> additionalConfiguration) {
@@ -102,7 +134,7 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
 
     @Override
     public void setResourceBaseDir(File resourceBaseDir) {
-        // no resource processing necessary
+        this.resourceBaseDir = resourceBaseDir;
     }
 
     @Override
@@ -116,29 +148,51 @@ public final class CodeGeneratorImpl implements CodeGenerator, BuildContextAware
         this.buildContext = Preconditions.checkNotNull(buildContext);
     }
 
-    private File generateYangModuleInfo(File outputBaseDir, Module module, SchemaContext ctx) {
+    private Set<File> generateYangModuleInfo(File outputBaseDir, Module module, SchemaContext ctx,
+            Builder<String> providerSourceSet) {
+        Builder<File> generatedFiles = ImmutableSet.<File> builder();
+
         final YangModuleInfoTemplate template = new YangModuleInfoTemplate(module, ctx);
-        String generatedCode = template.generate();
-        if (generatedCode.isEmpty()) {
+        String moduleInfoSource = template.generate();
+        if (moduleInfoSource.isEmpty()) {
             throw new IllegalStateException("Generated code should not be empty!");
         }
+        String providerSource = template.generateModelProvider();
+
+        final File packageDir = GeneratorJavaFile.packageToDirectory(outputBaseDir,
+                BindingGeneratorUtil.moduleNamespaceToPackageName(module));
 
-        final File packageDir = GeneratorJavaFile.packageToDirectory(outputBaseDir, BindingGeneratorUtil.moduleNamespaceToPackageName(module));
+        generatedFiles.add(writeJavaSource(packageDir, BindingMapping.MODULE_INFO_CLASS_NAME, moduleInfoSource));
+        generatedFiles
+                .add(writeJavaSource(packageDir, BindingMapping.MODEL_BINDING_PROVIDER_CLASS_NAME, providerSource));
+        providerSourceSet.add(template.getModelBindingProviderName());
+
+        return generatedFiles.build();
+
+    }
 
-        final File file = new File(packageDir, "$YangModuleInfoImpl.java");
+    private File writeJavaSource(File packageDir, String className, String source) {
+        if (!packageDir.exists()) {
+            packageDir.mkdirs();
+        }
+        final File file = new File(packageDir, className + ".java");
+        writeFile(file, source);
+        return file;
+    }
+
+    private File writeFile(File file, String source) {
         try (final OutputStream stream = buildContext.newFileOutputStream(file)) {
             try (final Writer fw = new OutputStreamWriter(stream)) {
                 try (final BufferedWriter bw = new BufferedWriter(fw)) {
-                    bw.write(generatedCode);
+                    bw.write(source);
                 }
             } catch (Exception e) {
-                // TODO handle exception
+                logger.error("Could not write file: {}",file,e);
             }
         } catch (Exception e) {
-            // TODO handle exception
+            logger.error("Could not create file: {}",file,e);
         }
         return file;
-
     }
 
 }