You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/01/30 00:03:01 UTC
svn commit: r616553 - in
/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies:
Dependencies.java renderer/DependenciesRenderer.java
Author: vsiveton
Date: Tue Jan 29 15:02:59 2008
New Revision: 616553
URL: http://svn.apache.org/viewvc?rev=616553&view=rev
Log:
MPIR-81: DependenciesRenderer logs Exception when it can't find the artifact being built!!!
o log a warning if the artifact of the current project has never been deployed
Modified:
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
Modified: maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java?rev=616553&r1=616552&r2=616553&view=diff
==============================================================================
--- maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java (original)
+++ maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java Tue Jan 29 15:02:59 2008
@@ -41,6 +41,8 @@
*/
public class Dependencies
{
+ private final MavenProject project;
+
private final List projectDependencies;
private final DependencyTree dependencyTree;
@@ -49,6 +51,7 @@
public Dependencies( MavenProject project, DependencyTree dependencyTree, JarClassesAnalysis classesAnalyzer )
{
+ this.project = project;
this.dependencyTree = dependencyTree;
this.projectDependencies = dependencyTree.getRootNode().getChildren();
this.classesAnalyzer = classesAnalyzer;
@@ -65,6 +68,16 @@
}
mapArtifactFiles( dependencyTree.getRootNode(), projectMap );
+ }
+
+ /**
+ * Getter for the project
+ *
+ * @return the project
+ */
+ public MavenProject getProject()
+ {
+ return project;
}
public boolean hasDependencies()
Modified: maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java?rev=616553&r1=616552&r2=616553&view=diff
==============================================================================
--- maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java (original)
+++ maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java Tue Jan 29 15:02:59 2008
@@ -345,7 +345,17 @@
}
catch ( ArtifactNotFoundException e )
{
- log.error( "Artifact: " + artifact.getId() + " has no file.", e );
+ if ( ( dependencies.getProject().getGroupId().equals( artifact.getGroupId() ) )
+ && ( dependencies.getProject().getArtifactId().equals( artifact.getArtifactId() ) )
+ && ( dependencies.getProject().getVersion().equals( artifact.getVersion() ) ) )
+ {
+ log.warn( "The artifact of this project has never been deployed." );
+ }
+ else
+ {
+ log.error( "Artifact: " + artifact.getId() + " has no file.", e );
+ }
+
continue;
}
}