You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@creadur.apache.org by sebb <se...@gmail.com> on 2014/08/12 02:40:57 UTC

Re: svn commit: r1617370 - in /creadur/rat/trunk: ./ apache-rat-plugin/ apache-rat-plugin/src/main/java/org/apache/rat/mp/ apache-rat-plugin/src/test/java/org/apache/rat/mp/

On 11 August 2014 22:40,  <po...@apache.org> wrote:
> Author: pottlinger
> Date: Mon Aug 11 21:40:26 2014
> New Revision: 1617370
>
> URL: http://svn.apache.org/r1617370
> Log:
> Refactoring mojo tests.
>
> -Replaced the way the MavenProject injected to a non-deprecated way (to get rid of the warning during the build).
> -Updated mavenInvokerPluginVersion and extracted it as a pom-property.
> -Extracted some RAT mojo integration test methods into a separate RatTestHelpers in order to find integration tests more easily after RAT-168 and the NPE when trying to forge 0.11.
>

[]

> ==============================================================================
> --- creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/AbstractRatMojo.java (original)
> +++ creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/AbstractRatMojo.java Mon Aug 11 21:40:26 2014
> @@ -198,9 +198,9 @@ public abstract class AbstractRatMojo ex
>         private boolean excludeSubProjects;
>
>         /**
> -        * Holds the maven-internal project to allow resolution of artifact properties during mojo runs.
> -        */
> -       @Component
> +    * Holds the maven-internal project to allow resolution of artifact properties during mojo runs.
> +    */
> +       @Parameter(property = "project", required = true, readonly = true)

So the NPE was caused by using

          @Parameter(name = "project",

i.e. the wrong attribute name?

>         private MavenProject project;
>
>         /**


[]