Return-Path: 
 <dev-return-21724-apmail-harmony-dev-archive=harmony.apache.org@harmony.apache.org>
Delivered-To: apmail-harmony-dev-archive@www.apache.org
Received: (qmail 32062 invoked from network); 14 Dec 2006 13:26:56 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 14 Dec 2006 13:26:56 -0000
Received: (qmail 66103 invoked by uid 500); 14 Dec 2006 13:26:57 -0000
Delivered-To: apmail-harmony-dev-archive@harmony.apache.org
Received: (qmail 66070 invoked by uid 500); 14 Dec 2006 13:26:57 -0000
Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@harmony.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@harmony.apache.org>
List-Post: <mailto:dev@harmony.apache.org>
List-Id: <dev.harmony.apache.org>
Reply-To: dev@harmony.apache.org
Delivered-To: mailing list dev@harmony.apache.org
Received: (qmail 66061 invoked by uid 99); 14 Dec 2006 13:26:57 -0000
Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133)
    by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 14 Dec 2006 05:26:57 -0800
X-ASF-Spam-Status: No, hits=2.0 required=10.0
	tests=HTML_MESSAGE,SPF_PASS
X-Spam-Check-By: apache.org
Received-SPF: pass (herse.apache.org: domain of ivavladimir@gmail.com
 designates 209.85.132.247 as permitted sender)
Received: from [209.85.132.247] (HELO an-out-0708.google.com) (209.85.132.247)
    by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 14 Dec 2006 05:26:47 -0800
Received: by an-out-0708.google.com with SMTP id b2so136156ana
        for <dev@harmony.apache.org>; Thu, 14 Dec 2006 05:26:26 -0800 (PST)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=V9sMeAKbG7kK2+Blufp3MPpTu4Iz/UIctk1f1D2y5xGqXmlwAXcYHoj2w74ZiGy+t3ex/ghz7X3IubTJNyDd54zWEgt+ALig2qSXXUpfN/NsMa5KXAh38r5dA4wIPFYv/mpH7CL8huUnFGhVWIutw8kpR8Sitg1KfWIlX4wTFn0=
Received: by 10.78.185.7 with SMTP id i7mr688201huf.1166102785151;
        Thu, 14 Dec 2006 05:26:25 -0800 (PST)
Received: by 10.78.145.6 with HTTP; Thu, 14 Dec 2006 05:26:24 -0800 (PST)
Message-ID: <7273946b0612140526x19e04005oe71cb90d2492f153@mail.gmail.com>
Date: Thu, 14 Dec 2006 19:26:24 +0600
From: "Vladimir Ivanov" <ivavladimir@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [classlib] new dependency (was: Re: [testing][crash handling] it
 will nice to add *any* crash handling to build system)
In-Reply-To: <2c9597b90612120936k758839beh5d69fe9419bfd929@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_149807_28901336.1166102784880"
References: <2c9597b90612091229i777aff36oefb3cc443fd856b9@mail.gmail.com>
	 <200612101525.kBAFPXx4022186@d06av02.portsmouth.uk.ibm.com>
	 <7273946b0612101949l4329a868s9e14122ec2cbbd60@mail.gmail.com>
	 <2c9597b90612111437y4f0ee551t99e59120b2b63083@mail.gmail.com>
	 <200612120933.kBC9Xhec001775@d06av02.portsmouth.uk.ibm.com>
	 <7273946b0612120154k71904b0ci72e3810214fd671c@mail.gmail.com>
	 <2c9597b90612120936k758839beh5d69fe9419bfd929@mail.gmail.com>
X-Virus-Checked: Checked by ClamAV on apache.org

------=_Part_149807_28901336.1166102784880
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

It is OK for me. It is better than current state and can be easily improved
if we will need more than it
:)

 Thanks, Vladimir


On 12/12/06, Alexei Zakharov <alexei.zakharov@gmail.com> wrote:
>
> Guys,
>
> Please look at the following code. I probably did the best I can with
> core tasks:
>
> ---
>    <target name="-process-empty-reports">
>       <property name="per.crashed.tests.file"
>                 value="${tests.output}${file.separator}TEST-crashed.xml"/>
>
>       <property name="per.header"
>                 value="&lt;?xml version='1.0' encoding='UTF-8'
> ?&gt;&lt;testsuite errors='1' name='vmcrash.CrashedTests'
> time='0'&gt;"/>
>       <property name="per.prefix"
>                 value="&lt;testcase classname='"/>
>       <property name="per.middle" value="' name='"/>
>       <property name=" per.suffix"
>                 value="' time='0.0'&gt;&lt;error message='VM
> crash?'/&gt;&lt;/testcase&gt;"/>
>       <property name="per.footer" value="&lt;/testsuite&gt;"/>
>
>       <fileset id="empty.test.reports" dir="${tests.output}">
>         <patternset>
>           <include name="TEST-*.xml"/>
>         </patternset>
>         <size value="0" when="equal"/>
>       </fileset>
>
>       <pathconvert refid="empty.test.reports" pathsep="@"
>                    property="crashed.tests"/>
>       <echo message="${crashed.tests }" file="${per.crashed.tests.file}"/>
>
>       <replaceregexp file="${per.crashed.tests.file}" flags="g">
>            <regexp pattern="[^@]*TEST-([^@]*)\.xml@?"/>
>            <substitution
> expression="${per.prefix}\1${per.middle}\1${per.suffix}"/>
>       </replaceregexp>
>       <replaceregexp file="${per.crashed.tests.file}" match="\A"
>                      replace="${per.header}"/>
>       <echo file="${per.crashed.tests.file}" append="yes"
>             message="${per.footer}"/>
>    </target>
> ---
>
> Drawbacks:
> - crashed tests are listed under special stub package called
> "vmcrash", so it is separated from other tests (not crashed) from the
> same package
> - actual number of crashes cannot be calculated and now always set to one
>
> But if Mark insists on core-task-only solution then it seems we have
> nothing more than to apply the above code (or something similar to
> it).
>
> Regards,
>
> 2006/12/12, Vladimir Ivanov < ivavladimir@gmail.com>:
> > On 12/12/06, Mark Hindess <mark.hindess@googlemail.com> wrote:
> > >
> > >
> > > I can't see how to do exactly what you want, but perhaps it would be
> > > sufficient to use the echo task to create a "TEST-crashed.xml" file
> with
> > > the list of tests (pathconvert with dirsep="${line.separator}") as the
> > > error message?
> >
> >
> >
> > I think it will be enough. In general, error and output streams are
> missed
> > for these tests so we can have one failed test with a list of tests to
> > investigate.
> >
> >  Thanks, Vladimir
> >
> >
> >
> > > HTH,
> > > Mark.
> > >
> > > > Thanks!
> > > >
> > > > 2006/12/11, Vladimir Ivanov <ivavladimir@gmail.com >:
> > > > > Agree, this approach is better than additional dependency.
> > > > >
> > > > > Alexei, please, throw out my jira and use this code.
> > > > >
> > > > >  Thanks, Vladimir
> > > > >
> > > > >
> > > > > On 12/10/06, Mark Hindess <mark.hindess@googlemail.com> wrote:
> > > > > >
> > > > > >
> > > > > > Please don't add this dependency.  We should use something like:
> > > > > >
> > > > > > <project name="test">
> > > > > > <target name="check-for-empty" depends="-check-for-empty,
> > > fail-if-empty"
> > > > > > />
> > > > > > <target name="-check-for-empty">
> > > > > >    <fileset id="empty.test.reports" dir="test_report">
> > > > > >      <patternset>
> > > > > >        <include name="TEST-*.xml"/>
> > > > > >      </patternset>
> > > > > >      <size value="0" when="equal"/>
> > > > > >    </fileset>
> > > > > >    <pathconvert refid="empty.test.reports" property="
> crashed.tests"
> > > />
> > > > > >    <condition property=" tests.crashed">
> > > > > >      <not>
> > > > > >        <equals arg1="" arg2="${crashed.tests}" />
> > > > > >      </not>
> > > > > >    </condition>
> > > > > > </target>
> > > > > > <target name="fail-if-empty" if="${crashed.tests}">
> > > > > >    <fail>Crashed tests: "${ crashed.tests}"</fail>
> > > > > > </target>
> > > > > > </project>
> > > > > >
> > > > > >
> > > > > > -Mark.
> > > > > >
> > > > > > On 9 December 2006 at 23:29, "Alexei Zakharov" <
> > > alexei.zakharov@gmail.com
> > > > >
> > > > > > wrote:
> > > > > > > All,
> > > > > > >
> > > > > > > I am going to commit changes for HARMONY-2349
> ([classlib][testing]
> > > > > > > "detect crashes by the empty report files") now. However, I've
>
> > > > > > > realized that in order to do this I need to introduce a new
> > > dependency
> > > > > > > to our list of classlib dependencies - the ant-contrib
> package.
> > > > > > > Personally I don't see any harm in this. It is quite nice
> package,
> > > we
> > > > > > > can also use it for organizing iterative test runs for
> example.
> > > > > > > Moreover, there is no need to force users to copy it to
> > > ANT_HOME/lib
> > > > > > > or wherever since ant's taskdef target is capable to load
> JAR's
> > > > > > > dynamically. BTW, DRLVM build has been using ant-contrib from
> the
> > > > > > > beginning.
> > > > > > >
> > > > > > > But I'd like to hear from others. Is ant-contrib ok with
> everyone?
> > > > > > >
> > > > > > > Thanks,
> > > > > > >
> > > > > > > 2006/12/8, Vladimir Ivanov <ivavladimir@gmail.com>:
> > > > > > > > The last CC notification for 'Linux SUSE 9 gcc debug:
> > > classlib  over
> > > > > > ibmvm'
> > > > > > > > has failed status due to "There were test errors" but on the
> > > next lin
> > > > e
> > > > > > it
> > > > > > > > told "All Tests Passed".
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > Note, it happened due to test crash:
> > > > > > > >
> > > > > > > > [Fatal Error] :-1:-1: Premature end of file.
> > > > > > > >
> > > > > > > > The file
> > > > > > > >
> > > /export/viv/trunk/cc/projects/classlib/trunk/build/test_report/TEST-
> > > > > > > >
> > > org.apache.harmony.security.tests.java.security.IdentityScope2Test.xm
> > > > lis
> > > > > > > > not a valid XML document. It is possibly corrupted.
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > This crash change the build status to 'FAILED' but no report
>
> > > file wer
> > > > e
> > > > > > > > generated so junit- reporter miss this crash.
>
>
> --
> Alexei Zakharov,
> Intel ESSD
>

------=_Part_149807_28901336.1166102784880--