Return-Path: 
 <commits-return-52230-apmail-harmony-commits-archive=harmony.apache.org@harmony.apache.org>
Delivered-To: apmail-harmony-commits-archive@www.apache.org
Received: (qmail 84867 invoked from network); 21 Mar 2008 06:16:23 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 21 Mar 2008 06:16:23 -0000
Received: (qmail 53918 invoked by uid 500); 21 Mar 2008 06:16:21 -0000
Delivered-To: apmail-harmony-commits-archive@harmony.apache.org
Received: (qmail 53904 invoked by uid 500); 21 Mar 2008 06:16:21 -0000
Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@harmony.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@harmony.apache.org>
List-Post: <mailto:commits@harmony.apache.org>
List-Id: <commits.harmony.apache.org>
Reply-To: dev@harmony.apache.org
Delivered-To: mailing list commits@harmony.apache.org
Received: (qmail 53895 invoked by uid 99); 21 Mar 2008 06:16:21 -0000
Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136)
    by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 20 Mar 2008 23:16:21 -0700
X-ASF-Spam-Status: No, hits=-2000.0 required=10.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 21 Mar 2008 06:15:50 +0000
Received: by eris.apache.org (Postfix, from userid 65534)
	id F2E7E1A9832; Thu, 20 Mar 2008 23:16:01 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r639550 - in /harmony/enhanced/buildtest/trunk/infra:
 build.xml
 scripts/framework.xml
Date: Fri, 21 Mar 2008 06:16:00 -0000
To: commits@harmony.apache.org
From: smishura@apache.org
X-Mailer: svnmailer-1.0.8
Message-Id: <20080321061601.F2E7E1A9832@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: smishura
Date: Thu Mar 20 23:15:36 2008
New Revision: 639550

URL: http://svn.apache.org/viewvc?rev=639550&view=rev
Log:
* Add 'install' target
* move 'check-selected-names' target to build.xml
  and make it antcontrib free - it is absent during installation

Modified:
    harmony/enhanced/buildtest/trunk/infra/build.xml
    harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml

Modified: harmony/enhanced/buildtest/trunk/infra/build.xml
URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/trunk/infra/build.xml?rev=639550&r1=639549&r2=639550&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/build.xml (original)
+++ harmony/enhanced/buildtest/trunk/infra/build.xml Thu Mar 20 23:15:36 2008
@@ -67,6 +67,52 @@
     </propertyset>
 
     <!-- ================================================================== 
+                         Install
+         ================================================================== -->
+
+    <target name="install" depends="-check-selected-names" />
+
+    <target name="-check-selected-names">
+
+        <property name="tmp.file" value="tmp.txt" />
+
+        <echo file="${tmp.file}">tmp.file.prop=${test.suites}</echo>
+        <loadproperties srcfile="${tmp.file}">
+            <!-- test suites list may contain spaces -->
+            <filterchain>
+                <replaceregex pattern=" " replace="" flags="g" />
+            </filterchain>
+        </loadproperties>
+
+        <length property="test.suites.names.list.length"
+                string="${test.suites.names.list}" />
+        <length property="tmp.file.prop.length" string="${tmp.file.prop}" />
+
+        <fail>
+            <condition>
+                <and>
+                    <not>
+                        <!-- if an adaptor doesn't exists 
+                             test.suites.names.list doesn't contain it -->
+                        <equals arg1="${tmp.file.prop.length}"
+                                arg2="${test.suites.names.list.length}" />
+                    </not>
+                    <not>
+                        <!-- OK when '*' was specified -->
+                        <equals arg1="${test.suites}" arg2="*" />
+                    </not>
+                </and>
+            </condition>...
+------
+ERROR: Could not found adaptor for some specified test suites
+------ 
+       Please, check the names specified by test.suites property:
+               test.suites=${test.suites}
+        </fail>
+
+        <delete file="${tmp.file}" />
+    </target>
+    <!-- ================================================================== 
                          Framework Setup Functionality
          ================================================================== -->
     <target name="-init-setup">

Modified: harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml
URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml?rev=639550&r1=639549&r2=639550&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml (original)
+++ harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml Thu Mar 20 23:15:36 2008
@@ -52,38 +52,13 @@
     <!-- Sets up the framework -->
     <target name="setup-framework" 
             depends="load-depends, define-tasks, 
-                        check-selected-names, reorder-suites, -delete-temporal-properties"/>
+                        reorder-suites, -delete-temporal-properties"/>
 
     <!-- Initialize the framework for test execution -->
     <target name="init-framework" 
             depends="load-depends, define-tasks, load-suites-configuration, 
                         load-parameters, -delete-temporal-properties"/>
     
-    <!-- check the names of selected test suites -->
-    <target name="check-selected-names">
-        <if>
-            <not><equals arg1="${test.suites}" arg2="*"/></not>
-        <then>
-            <for list="${test.suites}" param="suite" trim="yes">
-            <sequential>
-                <fail>
-                    <condition>
-                        <not><available 
-                            file="${adaptors.dir}/@{suite}" type="dir"/>
-                        </not>
-                    </condition>...
-------
-ERROR: Could not found adaptor for '@{suite}' test suite. 
------- 
-Please, check the names specified by test.suites property:
-        test.suites=${test.suites}
-                </fail>
-            </sequential>
-            </for>
-        </then>
-        </if>
-    </target>
-
     <!-- load information about external dependencies locations
          obtained on setup stage -->
     <target name="load-depends">