Re: [PATCH] Maven2 Plugin

I have cheked your patch against the test project and it works fine. The 
@execute phase="generate-sources" started a forked lifecycle which is not 
necessary for the om goal. The @requiresDependencyResolution is also not 
necessary. In the attached patch it is disabled by changing the annotation 
to a unknown attribute, however It would be cleaner to remove it 
completely as in

 * @goal om
- * @execute phase="generate-sources"
- * @requiresDependencyResolution compile
+ * @phase generate-sources
 */

Please commit your patch, I can also do it if you prefer that.

    Thomas

On Sun, 17 Dec 2006, Jeffrey D. Brekke wrote:

> below is a small patch to the OM mojo in torque.  I haven't committed in a 
> while on the project, and am not *that* familiar with the plugins for maven2, 
> but there seems to be a typo which prevents the om goal to integrate with the 
> maven2 lifecycle.  All the other goals seem fine. This does work here when I 
> rebuild the plugin using these settings.
>
> Could someone review and try it?
>
> ----
>
> Index: src/main/java/org/apache/torque/mojo/OMMojo.java
> ===================================================================
> --- src/main/java/org/apache/torque/mojo/OMMojo.java    (revision 488127)
> +++ src/main/java/org/apache/torque/mojo/OMMojo.java    (working copy)
> @@ -32,8 +32,8 @@
>  * @author <a href="fischer@xxxxxxxxxxxx">Thomas Fischer</a>
>  *
>  * @goal om
> - * @execute phase="generate-sources"
> - * @requiresDependencyResolution compile
> + * @phase generate-sources
> + * @requires DependencyResolution compile
>  */
> public class OMMojo
>         extends DataModelTaskMojo
>
> --
> =====================================================================
> Jeffrey D. Brekke                                   jbrekke@xxxxxxxxx
> Wisconsin,  USA                                     brekke@xxxxxxxxxx
>                                                    ekkerbj@xxxxxxxxx
> http://www.bloglines.com/blog/jbrekke               ekkerbj@xxxxxxxxx
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: torque-dev-unsubscribe@xxxxxxxxxxxxx
> For additional commands, e-mail: torque-dev-help@xxxxxxxxxxxxx