*/
package org.opendaylight.yangtools.yang2sources.plugin;
+import java.util.HashSet;
+import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementReactor;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangInferencePipeline;
import com.google.common.annotations.VisibleForTesting;
+import com.google.common.base.Throwables;
import com.google.common.collect.Maps;
import java.io.Closeable;
import java.io.File;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
-import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
+import java.util.concurrent.ConcurrentMap;
+import org.apache.commons.io.IOUtils;
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.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.URLSchemaContextResolver;
import org.opendaylight.yangtools.yang.parser.util.NamedFileInputStream;
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 File META_INF_YANG_DIR = new File(META_INF_YANG_STRING);
- private final Log log;
private final File yangFilesRootDir;
private final File[] excludedFiles;
private final List<CodeGeneratorArg> codeGenerators;
private final MavenProject project;
private final boolean inspectDependencies;
private final BuildContext buildContext;
- private YangProvider yangProvider;
+ private final YangProvider yangProvider;
+ private final URLSchemaContextResolver resolver;
@VisibleForTesting
- YangToSourcesProcessor(Log log, File yangFilesRootDir, File[] excludedFiles, List<CodeGeneratorArg> codeGenerators,
+ YangToSourcesProcessor(File yangFilesRootDir, File[] excludedFiles, List<CodeGeneratorArg> codeGenerators,
MavenProject project, boolean inspectDependencies, YangProvider yangProvider) {
- this(new DefaultBuildContext(), log, yangFilesRootDir, excludedFiles, codeGenerators, project, inspectDependencies, yangProvider);
+ this(new DefaultBuildContext(), yangFilesRootDir, excludedFiles, codeGenerators, project,
+ inspectDependencies, yangProvider);
}
- private YangToSourcesProcessor(BuildContext buildContext, Log log, File yangFilesRootDir, File[] excludedFiles,
- List<CodeGeneratorArg> codeGenerators, MavenProject project, boolean inspectDependencies, YangProvider yangProvider) {
+ private YangToSourcesProcessor(BuildContext buildContext, File yangFilesRootDir, File[] excludedFiles,
+ List<CodeGeneratorArg> 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;
this.project = Util.checkNotNull(project, "project");
this.inspectDependencies = inspectDependencies;
this.yangProvider = yangProvider;
+ this.resolver = URLSchemaContextResolver.create("maven-plugin");
}
- YangToSourcesProcessor(BuildContext buildContext, Log log, File yangFilesRootDir, File[] excludedFiles, List<CodeGeneratorArg> codeGenerators,
- MavenProject project, boolean inspectDependencies) {
- this(log, yangFilesRootDir, excludedFiles, codeGenerators, project, inspectDependencies, new YangProvider());
+ YangToSourcesProcessor(BuildContext buildContext, File yangFilesRootDir, File[] excludedFiles,
+ List<CodeGeneratorArg> codeGenerators, MavenProject project, 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);
}
}
private ContextHolder processYang() throws MojoExecutionException {
- YangParserImpl parser = new YangParserImpl();
+ final CrossSourceStatementReactor.BuildAction reactor = YangInferencePipeline.RFC6020_REACTOR.newBuild();
+ SchemaContext resolveSchemaContext;
List<Closeable> 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<File> yangFilesInProject = Util.listFiles(yangFilesRootDir, excludedFiles, log);
+ final Collection<File> yangFilesInProject = Util.listFiles(yangFilesRootDir, excludedFiles);
+
+
final Collection<File> 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;
}
/*
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<InputStream> yangsInProject = new ArrayList<>();
+ final List<NamedFileInputStream> 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<InputStream> all = new ArrayList<>(yangsInProject);
+ List<InputStream> all = new ArrayList<>();
+ all.addAll(yangsInProject);
closeables.addAll(yangsInProject);
- Map<InputStream, Module> allYangModules;
/**
* Set contains all modules generated from input sources. Number of
Set<Module> projectYangModules;
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<InputStream> yangStreams = toStreamsWithoutDuplicates(dependentYangResult.getYangStreams());
+ all.addAll(yangStreams);
+ closeables.addAll(yangStreams);
}
- allYangModules = parser.parseYangModelsFromStreamsMapped(all);
+ resolveSchemaContext = reactor.buildEffective(all);
+ Set<Module> parsedAllYangModules = resolveSchemaContext.getModules();
projectYangModules = new HashSet<>();
- for (InputStream inProject : yangsInProject) {
- Module module = allYangModules.get(inProject);
- if (module != null) {
- projectYangModules.add(module);
+ for (Module module : parsedAllYangModules) {
+ final String path = module.getModuleSourcePath();
+ if (path != null) {
+ LOG.debug("Looking for source {}", path);
+ for (NamedFileInputStream is : yangsInProject) {
+ LOG.debug("In project destination {}", is.getFileDestination());
+ if (path.equals(is.getFileDestination())) {
+ LOG.debug("Module {} belongs to current project", module);
+ projectYangModules.add(module);
+ break;
+ }
+ }
}
}
-
} finally {
for (AutoCloseable closeable : closeables) {
closeable.close();
}
}
- Set<Module> 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));
+ LOG.info("{} {} files parsed from {}", LOG_PREFIX, Util.YANG_SUFFIX.toUpperCase(), yangsInProject);
return new ContextHolder(resolveSchemaContext, projectYangModules);
// 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);
+ }
+ }
+
+ private List<InputStream> toStreamsWithoutDuplicates(List<YangSourceFromDependency> list) throws IOException {
+ ConcurrentMap<String, YangSourceFromDependency> byContent = Maps.newConcurrentMap();
+
+ for (YangSourceFromDependency yangFromDependency : list) {
+ try (InputStream dataStream = yangFromDependency.openStream()) {
+ String contents = IOUtils.toString(dataStream);
+ byContent.putIfAbsent(contents, yangFromDependency);
+ }
+
+ }
+ List<InputStream> inputs = new ArrayList<>(byContent.size());
+ for (YangSourceFromDependency entry : byContent.values()) {
+ inputs.add(entry.openStream());
}
+ return inputs;
}
static class YangProvider {
+ private static final Logger LOG = LoggerFactory.getLogger(YangProvider.class);
+
+ void addYangsToMetaInf(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(project, yangFilesRootDir, excludedFiles, generatedYangDir);
- private static final String YANG_RESOURCE_DIR = "target" + File.separator + "yang";
+ // Also copy to the actual build output dir if different than "target". When running in
+ // Eclipse this can differ (eg "target-ide").
- void addYangsToMetaInf(Log log, MavenProject project, File yangFilesRootDir, File[] excludedFiles)
+ File actualGeneratedYangDir = new File(project.getBuild().getDirectory(),
+ CodeGeneratorArg.YANG_GENERATED_DIR.replace("target" + File.separator, ""));
+ if (!actualGeneratedYangDir.equals(generatedYangDir)) {
+ addYangsToMetaInf(project, yangFilesRootDir, excludedFiles, actualGeneratedYangDir);
+ }
+ }
+
+ private void addYangsToMetaInf(MavenProject project, File yangFilesRootDir,
+ File[] excludedFiles, File generatedYangDir)
throws MojoFailureException {
- File targetYangDir = new File(project.getBasedir(), YANG_RESOURCE_DIR);
+
+ File withMetaInf = new File(generatedYangDir, META_INF_YANG_STRING);
+ withMetaInf.mkdirs();
try {
- Collection<File> files = Util.listFiles(yangFilesRootDir, excludedFiles, null);
+ Collection<File> files = Util.listFiles(yangFilesRootDir, excludedFiles);
for (File file : files) {
- org.apache.commons.io.FileUtils.copyFile(file, new File(targetYangDir, file.getName()));
+ org.apache.commons.io.FileUtils.copyFile(file, new File(withMetaInf, file.getName()));
}
} catch (IOException e) {
- String message = "Unable to list yang files into resource folder";
- log.warn(message, e);
- throw new MojoFailureException(message, e);
+ LOG.warn("Failed to generate files into root {}", yangFilesRootDir, e);
+ throw new MojoFailureException("Unable to list yang files into resource folder", e);
}
- setResource(targetYangDir, META_INF_YANG_STRING_JAR, project);
+ setResource(generatedYangDir, project);
- log.debug(Util.message("Yang files from: %s marked as resources: %s", LOG_PREFIX, yangFilesRootDir,
- META_INF_YANG_STRING_JAR));
+ LOG.debug("{} Yang files from: {} marked as resources: {}", LOG_PREFIX, yangFilesRootDir,
+ META_INF_YANG_STRING_JAR);
}
- private static void setResource(File targetYangDir, String targetPath, MavenProject project) {
+ private static void setResource(File targetYangDir, MavenProject project) {
Resource res = new Resource();
res.setDirectory(targetYangDir.getPath());
- if (targetPath != null) {
- res.setTargetPath(targetPath);
- }
project.addResource(res);
}
}
*/
private void generateSources(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;
}
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());
}
}
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 = Util.getInstance(codeGeneratorCfg.getCodeGeneratorClass(), BasicCodeGenerator.class);
+ LOG.info("{} Code generator instantiated from {}", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass());
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");
+ 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()));
+ 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, null, 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());
Collection<File> generated = g.generateSources(context.getContext(), outputDir, context.getYangModules());
- log.info(Util.message("Sources generated by %s: %s", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass(),
- generated));
+ LOG.info("{} Sources generated by {}: {}", LOG_PREFIX, codeGeneratorCfg.getCodeGeneratorClass(), generated);
}
}