Author: jhm
Date: Fri Mar  2 03:31:53 2007
New Revision: 513710

URL: http://svn.apache.org/viewvc?view=rev&rev=513710
Log:
<antversion>-TASK evaluates now atLeast and exactly attributes.

Modified:
    
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/AntVersion.java
    ant/core/trunk/src/tests/antunit/taskdefs/condition/antversion-test.xml

Modified: 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/AntVersion.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/AntVersion.java?view=diff&rev=513710&r1=513709&r2=513710
==============================================================================
--- 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/AntVersion.java 
(original)
+++ 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/condition/AntVersion.java 
Fri Mar  2 03:31:53 2007
@@ -41,7 +41,15 @@
         if (propertyname == null) {
             throw new BuildException("'property' must be set.");
         }
-        getProject().setNewProperty(propertyname, getVersion().toString());
+        if (atLeast!=null || exactly!=null) {
+            // If condition values are set, evaluate the condition
+            if (eval()) {
+                getProject().setNewProperty(propertyname, 
getVersion().toString());
+            }
+        } else {
+            // Raw task
+            getProject().setNewProperty(propertyname, getVersion().toString());
+        }
     }
 
     /**

Modified: 
ant/core/trunk/src/tests/antunit/taskdefs/condition/antversion-test.xml
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/tests/antunit/taskdefs/condition/antversion-test.xml?view=diff&rev=513710&r1=513709&r2=513710
==============================================================================
--- ant/core/trunk/src/tests/antunit/taskdefs/condition/antversion-test.xml 
(original)
+++ ant/core/trunk/src/tests/antunit/taskdefs/condition/antversion-test.xml Fri 
Mar  2 03:31:53 2007
@@ -23,8 +23,27 @@
     
   <target name="test-task">
     <antversion property="antversion"/>
-    <au:assertPropertyEquals name="antversion" value="1.7.1"/>
+    <au:assertPropertySet name="antversion" message="Property 'antversion' 
should be set."/>
+    <echo>AntVersion=${antversion}</echo>  
   </target>    
+    
+  <target name="test-property-conditional1">
+    <antversion property="antversion" atleast="2.0.0"/>
+    <au:assertTrue message="Property 'antversion' should not be set because 
this is not Ant 2.0.0+.">
+      <not>
+        <isset property="antversion"/>  
+      </not>  
+    </au:assertTrue>  
+  </target>
+    
+  <target name="test-property-conditional2">
+    <antversion property="antversion" atleast="1.7.0"/>
+    <au:assertTrue message="Property 'antversion' should be set because we 
should have Ant 1.7.0+ (${ant.version}).">
+      <isset property="antversion"/>  
+    </au:assertTrue>  
+  </target>
+    
+      
 
   <target name="all">
     <au:antunit>



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

Reply via email to