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="[EMAIL PROTECTED]">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 [EMAIL PROTECTED]
Wisconsin, USA [EMAIL PROTECTED]
[EMAIL PROTECTED]
http://www.bloglines.com/blog/jbrekke [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
