Return-Path: 
 <dev-return-20010-apmail-harmony-dev-archive=harmony.apache.org@harmony.apache.org>
Delivered-To: apmail-harmony-dev-archive@www.apache.org
Received: (qmail 69846 invoked from network); 24 Nov 2006 16:09:21 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 24 Nov 2006 16:09:21 -0000
Received: (qmail 79615 invoked by uid 500); 24 Nov 2006 16:09:19 -0000
Delivered-To: apmail-harmony-dev-archive@harmony.apache.org
Received: (qmail 79540 invoked by uid 500); 24 Nov 2006 16:09:19 -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 79435 invoked by uid 99); 24 Nov 2006 16:09:18 -0000
Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Nov 2006 08:09:18 -0800
X-ASF-Spam-Status: No, hits=1.4 required=10.0
	tests=SPF_HELO_PASS,SPF_NEUTRAL
X-Spam-Check-By: apache.org
Received-SPF: neutral (herse.apache.org: 216.86.168.178 is neither permitted
 nor denied by domain of geir@pobox.com)
Received: from [216.86.168.178] (HELO mxout-03.mxes.net) (216.86.168.178)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Nov 2006 08:09:04 -0800
Received: from [192.168.1.106] (unknown [67.86.14.213])
	(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by smtp.mxes.net (Postfix) with ESMTP id BCB9A51977
	for <dev@harmony.apache.org>; Fri, 24 Nov 2006 11:08:42 -0500 (EST)
Message-ID: <45671916.9010100@pobox.com>
Date: Fri, 24 Nov 2006 11:08:54 -0500
From: "Geir Magnusson Jr." <geir@pobox.com>
Reply-To: geir@pobox.com
User-Agent: Thunderbird 1.5.0.8 (Macintosh/20061025)
MIME-Version: 1.0
To: dev@harmony.apache.org
Subject: Re: [testing] test exclude list: can't we have incremental
 exclusions?
References: <8E389A5F2FEABA4CB1DEC35A25CB39CE78343E@mssmsx411>
	 <4565E071.2080609@pobox.com> <4565E239.6050803@pobox.com>
 <f944d83d0611232327h1df838uad9c6530b645025e@mail.gmail.com>
 <4566F772.9020007@pobox.com>
In-Reply-To: <4566F772.9020007@pobox.com>
Content-Type: text/plain; charset=KOI8-R; format=flowed
Content-Transfer-Encoding: 8bit
X-Virus-Checked: Checked by ClamAV on apache.org

As a point of process, ball is on your court.  I'm respecting your -1 
(although I wouldn't personally have been so forceful with a veto - and 
I'm not sure that this is really something that can be vetoed), but I 
expect us to discuss...

geir

Geir Magnusson Jr. wrote:
> 
> Alexey Varlamov wrote:
>> Geir,
>>
>> This sounds alarming - why do you need local exclude list? 
> 
> Because I may be testing something and I dont' want that test to be run 
> for some reason.
> 
>> This is
>> error prone, you might forget about locally excluded tests and then
>> commit improperly tested.
>> -1 until convincingly useful.
> 
> People are going to do it anyway - comment out things locally.  If I 
> screw up, and mask something, then everyone else is going to find my error.
> 
> I see no danger to this, and we make people's lives easier.
> 
> geir
> 
> 
>>
>> -- 
>> Alexey
>>
>> 24.11.06, Geir Magnusson Jr.<geir@pobox.com> �������(�):
>>> And while you're at it, how about making kind-and-gentle support for
>>> local excludes such that I can have a file
>>>
>>>     exclude.local
>>>
>>> which is my local exclusion list that
>>>
>>> a) will be svn-ignored and
>>>
>>> b) doesn't have to be there - so if a developer hasn't created the file,
>>> the build just keeps going...  I *think* that not having the file for an
>>> <excludesfile> entry will let the build keep going, but I'm not sure.
>>>
>>> geir
>>>
>>>
>>> Geir Magnusson Jr. wrote:
>>> > That works for me.  It will only increase the number of files if
>>> > platforms have bugs, but it will make for easier maintenance.
>>> >
>>> > We'll do the same in DRLVM too.
>>> >
>>> > geir
>>> >
>>> > Ivanov, Alexey A wrote:
>>> >> Hi everyone,
>>> >>
>>> >> Recently test exclude lists were removed from build.xml of the
>>> >> corresponding module, and there were added *six* files with excluded
>>> >> tests. These files contain almost the same list of files. The lists
>>> >> are identical for swing module. I found 2 differences for awt module
>>> >> (there are still about 50 files names listed in every of the exclude
>>> >> lists).
>>> >>
>>> >> Why can't we use one 'exclude.all' file to exclude tests which 
>>> fail on
>>> >> every platform? It's an obvious optimization.
>>> >> I've tested the approach of using several exclude list files on
>>> >> build.xml of swing module. It works just fine.
>>> >>
>>> >> Your comments?
>>> >>
>>> >> Regards,
>>> >> Alexey.
>>> >>
>>> >>
>>> >> ----- build.xml patch --------
>>> >> Index: build.xml
>>> >> ===================================================================
>>> >> --- build.xml   (revision 478584)
>>> >> +++ build.xml   (working copy)
>>> >> @@ -186,6 +186,7 @@
>>> >>
>>> >>                  <fileset dir="${hy.swing.src.test.api}/java/common">
>>> >>                      <include name="**/*Test*.java"/>
>>> >> +                       <excludesfile name="./make/exclude.all" />
>>> >>                      <excludesfile name="${exclude.file}" />
>>> >>                  </fileset>
>>> >>              </batchtest>
>>> >> ------------------------------
>>> >>
>>> >>
>>> >> --
>>> >> Alexey A. Ivanov
>>> >> Intel Enterprise Solutions Software Division
>>>