dev@glassfish.java.net

Re: PATCH bootstrap/maven.xml

From: Qingqing Ouyang <Qingqing.Ouyang_at_Sun.COM>
Date: Mon, 05 Dec 2005 19:55:30 -0600

Hi, Matthew:

Thanks much for the patch. Our regular build czar is on vacation right
now. I'll try to get to bring in your patch as soon as possible.

Thanks for your patience.
Q^2

Matthew L Daniel wrote:

>While I was trying to track down a compilation error, I found another
>peculiarity with the Maven/Ant hybrid build system. If one switches on
>the Maven debugging output (with -X to maven), that setting does not
>carry over into the forked Ant process.
>
>The attached patch against CVS HEAD fixes that.
>
> -- /v\atthew
>
>
>------------------------------------------------------------------------
>
>Index: maven.xml
>===================================================================
>RCS file: /cvs/glassfish/bootstrap/maven.xml,v
>retrieving revision 1.21
>diff -u -r1.21 maven.xml
>--- maven.xml 22 Nov 2005 19:31:22 -0000 1.21
>+++ maven.xml 5 Dec 2005 17:40:17 -0000
>@@ -290,6 +290,9 @@
> <pathelement path="${java.home}/../lib/tools.jar" />
> </classpath>
> <sysproperty key="ant.home" value="${glassfish.home}/lib/ant" />
>+ <j:if test='${maven.debugOn}'>
>+ <arg value='-debug' />
>+ </j:if>
> <arg line="-f ${module.dir}/build.xml -Dglassfish.maven=${glassfish.maven} -Dglassfish.home=${glassfish.home} ${ant.target} -Dbuild_id=${build_id}"/>
> </java>
>
>
>
>
>------------------------------------------------------------------------
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>
>