Return-Path: 
 <commits-return-54032-apmail-harmony-commits-archive=harmony.apache.org@harmony.apache.org>
Delivered-To: apmail-harmony-commits-archive@www.apache.org
Received: (qmail 68839 invoked from network); 25 Apr 2008 10:07:03 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 25 Apr 2008 10:07:03 -0000
Received: (qmail 40891 invoked by uid 500); 25 Apr 2008 10:07:04 -0000
Delivered-To: apmail-harmony-commits-archive@harmony.apache.org
Received: (qmail 40868 invoked by uid 500); 25 Apr 2008 10:07:04 -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 40857 invoked by uid 99); 25 Apr 2008 10:07:04 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 25 Apr 2008 03:07:04 -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, 25 Apr 2008 10:06:19 +0000
Received: by eris.apache.org (Postfix, from userid 65534)
	id 1A2A41A983E; Fri, 25 Apr 2008 03:06:39 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r651557 - in /harmony/enhanced/buildtest/trunk/infra:
 build.xml
 scripts/cc-exec-adaptor.xml scripts/framework.properties
 scripts/framework.xml
Date: Fri, 25 Apr 2008 10:06:38 -0000
To: commits@harmony.apache.org
From: smishura@apache.org
X-Mailer: svnmailer-1.0.8
Message-Id: <20080425100639.1A2A41A983E@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: smishura
Date: Fri Apr 25 03:06:34 2008
New Revision: 651557

URL: http://svn.apache.org/viewvc?rev=651557&view=rev
Log:
Remove tracking suites execution status via property file:
CruiseControl doesn't trigger the build in case if superior suite failed

Modified:
    harmony/enhanced/buildtest/trunk/infra/build.xml
    harmony/enhanced/buildtest/trunk/infra/scripts/cc-exec-adaptor.xml
    harmony/enhanced/buildtest/trunk/infra/scripts/framework.properties
    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=651557&r1=651556&r2=651557&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/build.xml (original)
+++ harmony/enhanced/buildtest/trunk/infra/build.xml Fri Apr 25 03:06:34 2008
@@ -390,7 +390,6 @@
             </if>
             <mkdir dir="${cc.work.dir}" />
             <mkdir dir="${cc.log.dir}" />
-            <touch file="${cc.execution.status.file}" />
         </sequential>
     </target>
 

Modified: harmony/enhanced/buildtest/trunk/infra/scripts/cc-exec-adaptor.xml
URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/trunk/infra/scripts/cc-exec-adaptor.xml?rev=651557&r1=651556&r2=651557&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/scripts/cc-exec-adaptor.xml (original)
+++ harmony/enhanced/buildtest/trunk/infra/scripts/cc-exec-adaptor.xml Fri Apr 25 03:06:34 2008
@@ -29,9 +29,6 @@
     <property name="suite.adaptor.dir" 
               value="${root.dir}/adaptors/${suite.name}"/>
     
-    <!-- load test suites execution statuses -->
-    <property file="${cc.execution.status.file}"/>
-
     <property name="suite.adaptor.dir" 
               value="${root.dir}/adaptors/${suite.name}"/>
     
@@ -39,41 +36,6 @@
     <target name="cc-run" depends="define-ant-contrib">
         <log message="Test Suite '${suite.name}' execution. Repository updated: ${repository.updated}"/>
 
-        <!-- clear previous status -->
-        <replaceregexp file="${cc.execution.status.file}"
-                       match="^${suite.name}\.parameters\..*$"
-                       replace=""
-                       byline="true"/>
-
-        <!-- FIXME: is this really required?
-             CruiseControl doesn't trigger the
-             build in case of failed buildstatus
-        
-        <check-superior-failures
-            suite="${suite.name}"
-            check-target="run"
-            property="failed.superiors"
-        />
-        -->
-        <if>
-            <isset property="failed.superiors"/>
-        <then>
-            <!-- write 'skipped' status -->
-            <propertyfile file="${cc.execution.status.file}"
-                          comment="${cc.execution.status.file.header}.">
-                <entry key="${suite.name}.parameters.run.skipped" value="true"/>
-            </propertyfile>
-            <echo level="warning">
-Test Suite '${suite.name}' was not executed (SKIPPED) 
-as the following preceding suites on which '${suite.name}' depends upon
-failed to run or have been skipped:
-    ${failed.superiors}
-</echo>
-            <fail message="Execution SKIPPED (failed superiors: ${failed.superiors})."/>
-
-        </then>
-        </if>
-
         <echo file="${cc.execution.log.file}" 
               message="&lt;html>&lt;body>"/>
         <echo file="${cc.execution.log.file}" append="true"
@@ -100,11 +62,6 @@
 ${exception}
 </echo>
             <record name="${cc.execution.log.file}" action="stop"/>
-            <!-- write 'failed' status -->
-            <propertyfile file="${cc.execution.status.file}"
-                          comment="${cc.execution.status.file.header}.">
-                <entry key="${suite.name}.parameters.run.failed" value="true"/>
-            </propertyfile>
         </catch>
         </trycatch>
         <echo file="${cc.execution.log.file}" append="true" 

Modified: harmony/enhanced/buildtest/trunk/infra/scripts/framework.properties
URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/trunk/infra/scripts/framework.properties?rev=651557&r1=651556&r2=651557&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/scripts/framework.properties (original)
+++ harmony/enhanced/buildtest/trunk/infra/scripts/framework.properties Fri Apr 25 03:06:34 2008
@@ -12,8 +12,6 @@
 cc.project.template=${scripts.dir}/templates/cc-project-template.xml
 cc.log.dir=${cc.work.dir}/logs
 cc.project.execution.timeout=5000000
-cc.execution.status.file=${cc.work.dir}/execution.status
-cc.execution.status.file.header=\ Autogenerated file used for Test Suites execution status checking.
 cc.execution.log.file=${cc.work.dir}/execution_log.html
 
 external.libs.dir=${build.dir}/libs

Modified: harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml
URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml?rev=651557&r1=651556&r2=651557&view=diff
==============================================================================
--- harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml (original)
+++ harmony/enhanced/buildtest/trunk/infra/scripts/framework.xml Fri Apr 25 03:06:34 2008
@@ -133,35 +133,6 @@
         </sequential>
     </macrodef>
 
-    <!-- check for the failures of the superior suites and place their
-         names (if any) into specified property -->
-    <macrodef name="check-superior-failures">
-        <attribute name="suite"/>
-        <attribute name="check-target"/>
-        <attribute name="property" default="@{suite}.has.failed.superior"/>
-        <sequential>
-            <var name="@{property}" unset="true"/>
-
-            <for list="${@{suite}.parameters.depends}" param="superior">
-            <sequential>
-                <log message="Check for '@{check-target}' failure of @{superior} = ${@{superior}.parameters.@{check-target}.failed}"/>
-                <if>
-                    <or>
-                        <istrue value="${@{superior}.parameters.@{check-target}.failed}"/>
-                        <istrue value="${@{superior}.parameters.@{check-target}.skipped}"/>
-                    </or>
-                <then>
-                    <!-- empty value as initial -->
-                    <property name="@{property}" value=""/>
-                    <var name="@{property}" value="${@{property}}@{superior} "/>
-                </then>
-                </if>
-            </sequential>
-            </for>
-            <log message="Result: @{property} = '${@{property}}'"/>
-        </sequential>
-    </macrodef>
-
     <!-- removes svn locks leaved after unsuccessful checkout or update-->
     <macrodef name="svn-cleanup">
         <attribute name="trunk"/>