You should apply the "if" to the test target
and not the all target.

i.e.:
<target name="test-not-enough-space-human" if="jdk.available">

Peter

On 12/13/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Author: kevj
Date: Wed Dec 13 01:55:55 2006
New Revision: 486570

URL: http://svn.apache.org/viewvc?view=rev&rev=486570
Log:
-fix hasfreespace-test to avoid problems on pre jdk6 vms

Modified:
    ant/core/trunk/src/tests/antunit/taskdefs/condition/hasfreespace-test.xml

Modified: 
ant/core/trunk/src/tests/antunit/taskdefs/condition/hasfreespace-test.xml
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/tests/antunit/taskdefs/condition/hasfreespace-test.xml?view=diff&rev=486570&r1=486569&r2=486570
==============================================================================
--- ant/core/trunk/src/tests/antunit/taskdefs/condition/hasfreespace-test.xml 
(original)
+++ ant/core/trunk/src/tests/antunit/taskdefs/condition/hasfreespace-test.xml 
Wed Dec 13 01:55:55 2006
@@ -1,6 +1,8 @@
 <?xml version="1.0"?>
 <project name="hasfreespace-test" default="all" basedir="." 
xmlns:au="antlib:org.apache.ant.antunit">

+  <available property="jdk6.available" classname="java.util.ServiceLoader"/>
+
   <target name="test-not-enough-space-human">
     <au:assertFalse>
       <hasfreespace partition="c:" needed="1P"/>
@@ -26,7 +28,7 @@
     </au:assertTrue>
   </target>

-  <target name="all">
+  <target name="all" if="jdk6.available">
     <au:antunit>
       <fileset file="${ant.file}"/>
       <au:plainlistener/>



---------------------------------------------------------------------
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]

Reply via email to