On 10 August 2014 22:15,  <[email protected]> wrote:
> Author: pottlinger
> Date: Sun Aug 10 21:15:56 2014
> New Revision: 1617162
>
> URL: http://svn.apache.org/r1617162
> Log:
> 0.11: NPE-fix
>
> Always ensuring that a list of repositories is existing in case its null.
> Possibly this is a change in the API induced by dependency updates.
>
> Stacktrace was:
> Caused by: java.lang.NullPointerException
>     at 
> org.apache.rat.mp.RatReportMojo.getSkinArtifactFile(RatReportMojo.java:114)
>     at org.apache.rat.mp.RatReportMojo.execute(RatReportMojo.java:204)
>     at 
> org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
>
>
> Modified:
>     
> creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/RatReportMojo.java
>
> Modified: 
> creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/RatReportMojo.java
> URL: 
> http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/RatReportMojo.java?rev=1617162&r1=1617161&r2=1617162&view=diff
> ==============================================================================
> --- 
> creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/RatReportMojo.java
>  (original)
> +++ 
> creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/RatReportMojo.java
>  Sun Aug 10 21:15:56 2014
> @@ -52,8 +52,10 @@ import java.io.IOException;
>  import java.io.PrintWriter;
>  import java.io.StringWriter;
>  import java.io.Writer;
> +import java.util.Collections;
>  import java.util.HashMap;
>  import java.util.Iterator;
> +import java.util.List;
>  import java.util.Locale;
>  import java.util.Map;
>  import java.util.ResourceBundle;
> @@ -111,7 +113,15 @@ public class RatReportMojo extends Abstr
>                  factory.createDependencyArtifact( skin.getGroupId(), 
> skin.getArtifactId(), versionSpec, "jar", null,
>                                                    null );
>
> -            resolver.resolve( artifact, 
> getProject().getRemoteArtifactRepositories(), localRepository );
> +            // FIXME Try to avoid NPE / found during 0.11-build checks
> +            // API is so old/without type information
> +                       @SuppressWarnings("unchecked")
> +                       List<ArtifactRepository> remoteArtifactRepositories = 
> getProject().getRemoteArtifactRepositories();

Fails with NPE - likely because getProject() is null - same issue as
I'd already discovered.

> +            if(remoteArtifactRepositories == null) {
> +               remoteArtifactRepositories = Collections.emptyList();
> +            }
> +
> +                       resolver.resolve( artifact, 
> remoteArtifactRepositories, localRepository );
>          }
>          catch ( InvalidVersionSpecificationException e )
>          {
>
>

Reply via email to