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 );