Revision: 6239
Author: [email protected]
Date: Mon Sep 28 12:14:04 2009
Log: Ant fix; junit.stop.build should be set by the junit.failure property,  
not whatever property is named by its ${junit.failure} value.  Up-merge  
from r6234.

Review by: amitmanjhi
http://code.google.com/p/google-web-toolkit/source/detail?r=6239

Modified:
  /trunk/common.ant.xml

=======================================
--- /trunk/common.ant.xml       Thu Sep  3 12:14:51 2009
+++ /trunk/common.ant.xml       Mon Sep 28 12:14:04 2009
@@ -199,7 +199,6 @@
          <isset property="emma.enabled" />
        </condition>

-      <echo message="${javac.out} ${javac.junit.out}" />
        <junit dir="@{test.out}" fork="yes" printsummary="yes"
               failureproperty="junit.failure" tempdir="@{test.out}">
          <jvmarg line="${junit.platform.args}" />
@@ -249,7 +248,7 @@
        <condition property="junit.stop.build" value="true">
          <and>
            <istrue value="@{haltonfailure}"/>
-          <isset property="${junit.failure}"/>
+          <isset property="junit.failure"/>
          </and>
        </condition>
        <fail message="One or more junit tests failed" if="junit.stop.build"  
/>

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to