> -----Original Message-----
> From: Felipe Leme [mailto:[EMAIL PROTECTED]
> Sent: mardi 11 janvier 2005 01:57
> To: Cactus Developers List
> Subject: RE: cvs commit: jakarta-cactus/integration/maven plugin.jelly
> 
> On Mon, 2005-01-10 at 19:32, Vincent Massol wrote:
> > Cool. Does it work if I don't define any value for
> cactus.jboss3x.jvmArgs.
> 
> Yes, it  works, as the Ant task checks for it:
> 
>        // pass arguments to the JVM
>         if (this.jvmArgs != null)
>         {
>             getLog().trace(
>                 "Passing arguments to the container JVM: " +
> this.jvmArgs);
>             java.createJvmarg().setLine(this.jvmArgs);
>         }

Ahah... My bad. I thought you were introducing a new property. I should have
known better! :-)

> 
> > Felipe, do you think you could also document this new property in the
> > plugin.properties file, in the xdocs/properties.xml one and in the
> > xdocs/changes.xml one?
> 
> Actually, the property is already documented, as I have fixed this bug
> some weeks ago. But in my original fix I just added the jvmArgs in one
> call of each container (on cactus:test-war, I think), I forgot to add it
> on the other calls (like cactus:test-ear).
> 
> So this commit was just a complement of the first one (just today a
> friend of mine realized the argument was not being passed to the
> cactus:cactus-ear goal :-(

Thanks Felipe and sorry for my confusion...

-Vincent




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to