X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=yang%2Fyang-maven-plugin%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fyangtools%2Fyang2sources%2Fplugin%2FYangToSourcesProcessor.java;h=3389404a8071de1c4dccf474849f056e21d2c4c2;hb=f977321197fb8b6cd5de576483f413ab6742766b;hp=c6c0badda738bebedccbce0340a91ece03b56cbc;hpb=a67638e90f593d458ec185ef02487e95292d99ed;p=yangtools.git diff --git a/yang/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/YangToSourcesProcessor.java b/yang/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/YangToSourcesProcessor.java index c6c0badda7..3389404a80 100644 --- a/yang/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/YangToSourcesProcessor.java +++ b/yang/yang-maven-plugin/src/main/java/org/opendaylight/yangtools/yang2sources/plugin/YangToSourcesProcessor.java @@ -8,103 +8,136 @@ package org.opendaylight.yangtools.yang2sources.plugin; import com.google.common.annotations.VisibleForTesting; -import com.google.common.collect.Maps; +import com.google.common.base.Preconditions; +import com.google.common.base.Throwables; +import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableSet; +import com.google.common.io.CharStreams; import java.io.Closeable; import java.io.File; import java.io.IOException; import java.io.InputStream; +import java.io.Reader; +import java.nio.charset.StandardCharsets; import java.util.ArrayList; import java.util.Collection; -import java.util.Collections; +import java.util.HashMap; import java.util.HashSet; import java.util.List; import java.util.Map; import java.util.Set; -import org.apache.maven.model.Resource; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.MojoFailureException; -import org.apache.maven.plugin.logging.Log; import org.apache.maven.project.MavenProject; +import org.codehaus.plexus.util.FileUtils; import org.opendaylight.yangtools.yang.model.api.Module; import org.opendaylight.yangtools.yang.model.api.SchemaContext; -import org.opendaylight.yangtools.yang.parser.impl.YangParserImpl; +import org.opendaylight.yangtools.yang.parser.repo.YangTextSchemaContextResolver; import org.opendaylight.yangtools.yang.parser.util.NamedFileInputStream; +import org.opendaylight.yangtools.yang.test.util.YangParserTestUtils; import org.opendaylight.yangtools.yang2sources.plugin.ConfigArg.CodeGeneratorArg; import org.opendaylight.yangtools.yang2sources.plugin.Util.ContextHolder; import org.opendaylight.yangtools.yang2sources.plugin.Util.YangsInZipsResult; +import org.opendaylight.yangtools.yang2sources.spi.BasicCodeGenerator; import org.opendaylight.yangtools.yang2sources.spi.BuildContextAware; -import org.opendaylight.yangtools.yang2sources.spi.CodeGenerator; +import org.opendaylight.yangtools.yang2sources.spi.MavenProjectAware; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import org.sonatype.plexus.build.incremental.BuildContext; import org.sonatype.plexus.build.incremental.DefaultBuildContext; class YangToSourcesProcessor { + private static final Logger LOG = LoggerFactory.getLogger(YangToSourcesProcessor.class); + static final String LOG_PREFIX = "yang-to-sources:"; static final String META_INF_YANG_STRING = "META-INF" + File.separator + "yang"; static final String META_INF_YANG_STRING_JAR = "META-INF" + "/" + "yang"; + static final String META_INF_YANG_SERVICES_STRING_JAR = "META-INF" + "/" + "services"; - private final Log log; private final File yangFilesRootDir; - private final File[] excludedFiles; + private final Set excludedFiles; private final List codeGenerators; private final MavenProject project; private final boolean inspectDependencies; private final BuildContext buildContext; private final YangProvider yangProvider; + private final YangTextSchemaContextResolver resolver; @VisibleForTesting - YangToSourcesProcessor(Log log, File yangFilesRootDir, File[] excludedFiles, List codeGenerators, - MavenProject project, boolean inspectDependencies, YangProvider yangProvider) { - this(new DefaultBuildContext(), log, yangFilesRootDir, excludedFiles, codeGenerators, project, inspectDependencies, yangProvider); + YangToSourcesProcessor(final File yangFilesRootDir, final Collection excludedFiles, + final List codeGenerators, final MavenProject project, final boolean inspectDependencies, + final YangProvider yangProvider) { + this(new DefaultBuildContext(), yangFilesRootDir, excludedFiles, codeGenerators, project, + inspectDependencies, yangProvider); } - private YangToSourcesProcessor(BuildContext buildContext, Log log, File yangFilesRootDir, File[] excludedFiles, - List codeGenerators, MavenProject project, boolean inspectDependencies, YangProvider yangProvider) { - this.buildContext = Util.checkNotNull(buildContext, "buildContext"); - this.log = Util.checkNotNull(log, "log"); - this.yangFilesRootDir = Util.checkNotNull(yangFilesRootDir, "yangFilesRootDir"); - this.excludedFiles = new File[excludedFiles.length]; - int i = 0; - for (File file : excludedFiles) { - this.excludedFiles[i++] = new File(file.getPath()); - } - this.codeGenerators = Collections.unmodifiableList(Util.checkNotNull(codeGenerators, "codeGenerators")); - this.project = Util.checkNotNull(project, "project"); + private YangToSourcesProcessor(final BuildContext buildContext, final File yangFilesRootDir, + final Collection excludedFiles, final List codeGenerators, + final MavenProject project, final boolean inspectDependencies, final YangProvider yangProvider) { + this.buildContext = Preconditions.checkNotNull(buildContext, "buildContext"); + this.yangFilesRootDir = Preconditions.checkNotNull(yangFilesRootDir, "yangFilesRootDir"); + this.excludedFiles = ImmutableSet.copyOf(excludedFiles); + this.codeGenerators = ImmutableList.copyOf(codeGenerators); + this.project = Preconditions.checkNotNull(project); this.inspectDependencies = inspectDependencies; this.yangProvider = yangProvider; + this.resolver = YangTextSchemaContextResolver.create("maven-plugin"); } - YangToSourcesProcessor(BuildContext buildContext, Log log, File yangFilesRootDir, File[] excludedFiles, List codeGenerators, - MavenProject project, boolean inspectDependencies) { - this(log, yangFilesRootDir, excludedFiles, codeGenerators, project, inspectDependencies, new YangProvider()); + YangToSourcesProcessor(final BuildContext buildContext, final File yangFilesRootDir, + final Collection excludedFiles, final List codeGenerators, + final MavenProject project, final boolean inspectDependencies) { + this(yangFilesRootDir, excludedFiles, codeGenerators, project, inspectDependencies, new YangProvider()); } public void execute() throws MojoExecutionException, MojoFailureException { ContextHolder context = processYang(); if (context != null) { generateSources(context); - yangProvider.addYangsToMetaInf(log, project, yangFilesRootDir, excludedFiles); + yangProvider.addYangsToMetaInf(project, yangFilesRootDir, excludedFiles); + } + } + + void conditionalExecute(final boolean skip) throws MojoExecutionException, MojoFailureException { + if (skip) { + LOG.info("Skipping YANG code generation because property yang.skip is true"); + + // But manually add resources + // add META_INF/yang + yangProvider.addYangsToMetaInf(project, yangFilesRootDir, excludedFiles); + + // add META_INF/services + File generatedServicesDir = new GeneratedDirectories(project).getYangServicesDir(); + YangProvider.setResource(generatedServicesDir, project); + LOG.debug("{} Yang services files from: {} marked as resources: {}", LOG_PREFIX, generatedServicesDir, + META_INF_YANG_SERVICES_STRING_JAR); + + + } else { + execute(); } } private ContextHolder processYang() throws MojoExecutionException { - YangParserImpl parser = new YangParserImpl(); + SchemaContext resolveSchemaContext; List closeables = new ArrayList<>(); - log.info(Util.message("Inspecting %s", LOG_PREFIX, yangFilesRootDir)); + LOG.info("{} Inspecting {}", LOG_PREFIX, yangFilesRootDir); try { /* * Collect all files which affect YANG context. This includes all * files in current project and optionally any jars/files in the * dependencies. */ - final Collection yangFilesInProject = Util.listFiles(yangFilesRootDir, excludedFiles, log); + final Collection yangFilesInProject = Util.listFiles(yangFilesRootDir, excludedFiles); + final Collection allFiles = new ArrayList<>(yangFilesInProject); if (inspectDependencies) { - allFiles.addAll(Util.findYangFilesInDependencies(log, project)); + allFiles.addAll(Util.findYangFilesInDependencies(project)); } if (allFiles.isEmpty()) { - log.info(Util.message("No input files found", LOG_PREFIX)); - return null; + LOG.info("{} No input files found", LOG_PREFIX); + return null; } /* @@ -114,24 +147,26 @@ class YangToSourcesProcessor { boolean noChange = true; for (final File f : allFiles) { if (buildContext.hasDelta(f)) { - log.debug(Util.message("buildContext %s indicates %s changed, forcing regeneration", LOG_PREFIX, buildContext, f)); + LOG.debug("{} buildContext {} indicates {} changed, forcing regeneration", LOG_PREFIX, + buildContext, f); noChange = false; } } if (noChange) { - log.info(Util.message("None of %s input files changed", LOG_PREFIX, allFiles.size())); + LOG.info("{} None of {} input files changed", LOG_PREFIX, allFiles.size()); return null; } - final List yangsInProject = new ArrayList<>(); + final List yangsInProject = new ArrayList<>(); for (final File f : yangFilesInProject) { + // FIXME: This is hack - normal path should be reported. yangsInProject.add(new NamedFileInputStream(f, META_INF_YANG_STRING + File.separator + f.getName())); } - List all = new ArrayList<>(yangsInProject); + List all = new ArrayList<>(); + all.addAll(yangsInProject); closeables.addAll(yangsInProject); - Map allYangModules; /** * Set contains all modules generated from input sources. Number of @@ -139,168 +174,177 @@ class YangToSourcesProcessor { * (parsed submodule's data are added to its parent module). Set * cannot contains null values. */ - Set projectYangModules; + final Set projectYangModules = new HashSet<>(); + final Set projectYangFiles = new HashSet<>(); try { if (inspectDependencies) { - YangsInZipsResult dependentYangResult = Util.findYangFilesInDependenciesAsStream(log, project); + YangsInZipsResult dependentYangResult = Util.findYangFilesInDependenciesAsStream(project); Closeable dependentYangResult1 = dependentYangResult; closeables.add(dependentYangResult1); - all.addAll(dependentYangResult.getYangStreams()); + List yangStreams = toStreamsWithoutDuplicates(dependentYangResult.getYangStreams()); + all.addAll(yangStreams); + closeables.addAll(yangStreams); } - allYangModules = parser.parseYangModelsFromStreamsMapped(all); + resolveSchemaContext = YangParserTestUtils.parseYangStreams(all); - projectYangModules = new HashSet<>(); - for (InputStream inProject : yangsInProject) { - Module module = allYangModules.get(inProject); - if (module != null) { + Set parsedAllYangModules = resolveSchemaContext.getModules(); + for (Module module : parsedAllYangModules) { + if (containedInFiles(yangsInProject, module)) { + LOG.debug("Module {} belongs to current project", module); projectYangModules.add(module); + projectYangFiles.add(module); + + for (Module sub : module.getSubmodules()) { + if (containedInFiles(yangsInProject, sub)) { + LOG.debug("Submodule {} belongs to current project", sub); + projectYangFiles.add(sub); + } else { + LOG.warn("Submodule {} not found in input files", sub); + } + } } } - } finally { for (AutoCloseable closeable : closeables) { closeable.close(); } } - Set parsedAllYangModules = new HashSet<>(allYangModules.values()); - SchemaContext resolveSchemaContext = parser.resolveSchemaContext(parsedAllYangModules); - log.info(Util.message("%s files parsed from %s", LOG_PREFIX, Util.YANG_SUFFIX.toUpperCase(), yangsInProject)); - return new ContextHolder(resolveSchemaContext, projectYangModules); + LOG.info("{} {} files parsed from {}", LOG_PREFIX, Util.YANG_SUFFIX.toUpperCase(), yangsInProject); + LOG.debug("Project YANG files: {}", projectYangFiles); + + return new ContextHolder(resolveSchemaContext, projectYangModules, projectYangFiles); // MojoExecutionException is thrown since execution cannot continue } catch (Exception e) { - String message = Util.message("Unable to parse %s files from %s", LOG_PREFIX, Util.YANG_SUFFIX, - yangFilesRootDir); - log.error(message, e); - throw new MojoExecutionException(message, e); + LOG.error("{} Unable to parse {} files from {}", LOG_PREFIX, Util.YANG_SUFFIX, yangFilesRootDir, e); + Throwable rootCause = Throwables.getRootCause(e); + throw new MojoExecutionException(LOG_PREFIX + " Unable to parse " + Util.YANG_SUFFIX + " files from " + + yangFilesRootDir, rootCause); } } - static class YangProvider { - - void addYangsToMetaInf(Log log, MavenProject project, File yangFilesRootDir, File[] excludedFiles) - throws MojoFailureException { - - // copy project's src/main/yang/*.yang to target/generated-sources/yang/META-INF/yang/*.yang - - File generatedYangDir = new File(project.getBasedir(), CodeGeneratorArg.YANG_GENERATED_DIR); - addYangsToMetaInf(log, project, yangFilesRootDir, excludedFiles, generatedYangDir); - - // Also copy to the actual build output dir if different than "target". When running in - // Eclipse this can differ (eg "target-ide"). - - File actualGeneratedYangDir = new File(project.getBuild().getDirectory(), - CodeGeneratorArg.YANG_GENERATED_DIR.replace("target" + File.separator, "")); - if(!actualGeneratedYangDir.equals(generatedYangDir)) { - addYangsToMetaInf(log, project, yangFilesRootDir, excludedFiles, actualGeneratedYangDir); + private static boolean containedInFiles(final List files, final Module module) { + final String path = module.getModuleSourcePath(); + if (path != null) { + LOG.debug("Looking for source {}", path); + for (NamedFileInputStream is : files) { + LOG.debug("In project destination {}", is.getFileDestination()); + if (path.equals(is.getFileDestination())) { + return true; + } } } - private void addYangsToMetaInf(Log log, MavenProject project, File yangFilesRootDir, - File[] excludedFiles, File generatedYangDir) - throws MojoFailureException { + return false; + } - File withMetaInf = new File(generatedYangDir, META_INF_YANG_STRING); - withMetaInf.mkdirs(); + private static List toStreamsWithoutDuplicates(final List list) + throws IOException { + final Map byContent = new HashMap<>(); - try { - Collection files = Util.listFiles(yangFilesRootDir, excludedFiles, log); - for (File file : files) { - org.apache.commons.io.FileUtils.copyFile(file, new File(withMetaInf, file.getName())); - } + for (YangSourceFromDependency yangFromDependency : list) { + try (Reader reader = yangFromDependency.asCharSource(StandardCharsets.UTF_8).openStream()) { + final String contents = CharStreams.toString(reader); + byContent.putIfAbsent(contents, yangFromDependency); } catch (IOException e) { - log.warn(String.format("Failed to generate files into root %s", yangFilesRootDir), e); - throw new MojoFailureException("Unable to list yang files into resource folder", e); + throw new IOException("Exception when reading from: " + yangFromDependency.getDescription(), e); } - setResource(generatedYangDir, project); - - log.debug(Util.message("Yang files from: %s marked as resources: %s", LOG_PREFIX, yangFilesRootDir, - META_INF_YANG_STRING_JAR)); } - - private static void setResource(File targetYangDir, MavenProject project) { - Resource res = new Resource(); - res.setDirectory(targetYangDir.getPath()); - project.addResource(res); + List inputs = new ArrayList<>(byContent.size()); + for (YangSourceFromDependency entry : byContent.values()) { + inputs.add(entry.openStream()); } + return inputs; } /** * Call generate on every generator from plugin configuration */ - private void generateSources(ContextHolder context) throws MojoFailureException { + private void generateSources(final ContextHolder context) throws MojoFailureException { if (codeGenerators.size() == 0) { - log.warn(Util.message("No code generators provided", LOG_PREFIX)); + LOG.warn("{} No code generators provided", LOG_PREFIX); return; } - Map thrown = Maps.newHashMap(); + final Map thrown = new HashMap<>(); for (CodeGeneratorArg codeGenerator : codeGenerators) { try { generateSourcesWithOneGenerator(context, codeGenerator); } catch (Exception e) { // try other generators, exception will be thrown after - log.error( - Util.message("Unable to generate sources with %s generator", LOG_PREFIX, - codeGenerator.getCodeGeneratorClass()), e); + LOG.error("{} Unable to generate sources with {} generator", LOG_PREFIX, codeGenerator + .getCodeGeneratorClass(), e); thrown.put(codeGenerator.getCodeGeneratorClass(), e.getClass().getCanonicalName()); } } if (!thrown.isEmpty()) { - String message = Util.message( - "One or more code generators failed, including failed list(generatorClass=exception) %s", - LOG_PREFIX, thrown.toString()); - log.error(message); - throw new MojoFailureException(message); + String message = " One or more code generators failed, including failed list(generatorClass=exception) "; + LOG.error("{}" + message + "{}", LOG_PREFIX, thrown.toString()); + throw new MojoFailureException(LOG_PREFIX + message + thrown.toString()); } } /** * Instantiate generator from class and call required method */ - private void generateSourcesWithOneGenerator(ContextHolder context, CodeGeneratorArg codeGeneratorCfg) + private void generateSourcesWithOneGenerator(final ContextHolder context, final CodeGeneratorArg codeGeneratorCfg) throws ClassNotFoundException, InstantiationException, IllegalAccessException, IOException { codeGeneratorCfg.check(); - CodeGenerator g = Util.getInstance(codeGeneratorCfg.getCodeGeneratorClass(), CodeGenerator.class); - log.info(Util.message("Code generator instantiated from %s", LOG_PREFIX, - codeGeneratorCfg.getCodeGeneratorClass())); + BasicCodeGenerator g = getInstance(codeGeneratorCfg.getCodeGeneratorClass(), BasicCodeGenerator.class); + LOG.info("{} Code generator instantiated from {}", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass()); - File outputDir = codeGeneratorCfg.getOutputBaseDir(project); + final File outputDir = codeGeneratorCfg.getOutputBaseDir(project); - if (outputDir != null) { - project.addCompileSourceRoot(outputDir.getAbsolutePath()); - } else { - throw new NullPointerException("outputBaseDir is null. Please provide a valid outputBaseDir value in the pom.xml"); + if (outputDir == null) { + throw new NullPointerException("outputBaseDir is null. Please provide a valid outputBaseDir value in the " + + "pom.xml"); } - log.info(Util.message("Sources will be generated to %s", LOG_PREFIX, outputDir)); - log.debug(Util.message("Project root dir is %s", LOG_PREFIX, project.getBasedir())); - log.debug(Util.message("Additional configuration picked up for : %s: %s", LOG_PREFIX, - codeGeneratorCfg.getCodeGeneratorClass(), codeGeneratorCfg.getAdditionalConfiguration())); + project.addCompileSourceRoot(outputDir.getAbsolutePath()); + + LOG.info("{} Sources will be generated to {}", LOG_PREFIX, outputDir); + LOG.debug("{} Project root dir is {}", LOG_PREFIX, project.getBasedir()); + LOG.debug("{} Additional configuration picked up for : {}: {}", LOG_PREFIX, codeGeneratorCfg + .getCodeGeneratorClass(), codeGeneratorCfg.getAdditionalConfiguration()); if (g instanceof BuildContextAware) { ((BuildContextAware)g).setBuildContext(buildContext); } - g.setLog(log); - g.setMavenProject(project); + if (g instanceof MavenProjectAware) { + ((MavenProjectAware)g).setMavenProject(project); + } g.setAdditionalConfig(codeGeneratorCfg.getAdditionalConfiguration()); File resourceBaseDir = codeGeneratorCfg.getResourceBaseDir(project); YangProvider.setResource(resourceBaseDir, project); g.setResourceBaseDir(resourceBaseDir); - log.debug(Util.message("Folder: %s marked as resources for generator: %s", LOG_PREFIX, resourceBaseDir, - codeGeneratorCfg.getCodeGeneratorClass())); + LOG.debug("{} Folder: {} marked as resources for generator: {}", LOG_PREFIX, resourceBaseDir, + codeGeneratorCfg.getCodeGeneratorClass()); + + FileUtils.deleteDirectory(outputDir); + LOG.info("{} Succesfully deleted output directory {}", LOG_PREFIX, outputDir); - Collection generated = g.generateSources(context.getContext(), outputDir, context.getYangModules()); + Collection generated = g.generateSources(context.getContext(), outputDir, context.getYangModules(), + context::moduleToResourcePath); - log.info(Util.message("Sources generated by %s: %s", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass(), - generated)); + LOG.info("{} Sources generated by {}: {}", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass(), generated); } + /** + * Instantiate object from fully qualified class name + */ + private static T getInstance(final String codeGeneratorClass, final Class baseType) throws + ClassNotFoundException, InstantiationException, IllegalAccessException { + final Class clazz = Class.forName(codeGeneratorClass); + + Preconditions.checkArgument(baseType.isAssignableFrom(clazz), "Code generator %s has to implement %s", clazz, + baseType); + return baseType.cast(clazz.newInstance()); + } }