You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by tc...@apache.org on 2012/08/08 20:23:41 UTC

svn commit: r1370872 - /maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java

Author: tchemit
Date: Wed Aug  8 18:23:40 2012
New Revision: 1370872

URL: http://svn.apache.org/viewvc?rev=1370872&view=rev
Log:
MPIR-248 NPE while DependenciesReport

Do not treat system scope artifact with no file for dependencies report


Modified:
    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/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=1370872&r1=1370871&r2=1370872&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 Wed Aug  8 18:23:40 2012
@@ -1418,8 +1418,14 @@ public class DependenciesRenderer
         for ( Artifact artifact : artifacts )
         {
             // TODO site:run Why do we need to resolve this...
-            if ( artifact.getFile() == null && !Artifact.SCOPE_SYSTEM.equals( artifact.getScope() ) )
+            if ( artifact.getFile() == null )
             {
+                if ( Artifact.SCOPE_SYSTEM.equals( artifact.getScope() ) ) {
+
+                    // can not resolve system scope artifact file
+                    continue;
+                }
+
                 try
                 {
                     repoUtils.resolve( artifact );