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