Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: neutral (asf.osuosl.org: 64.74.244.70 is neither permitted nor
 denied by domain of geir@pobox.com)
Message-ID: <44D2113D.1030706@pobox.com>
Date: Thu, 03 Aug 2006 11:07:41 -0400
From: Geir Magnusson Jr <geir@pobox.com>
Reply-To: geir@pobox.com
User-Agent: Thunderbird 1.5.0.5 (Windows/20060719)
MIME-Version: 1.0
To: harmony-dev@incubator.apache.org
Subject: Re: [continuum] BUILD FAILURE: Classlib/win.ia32 Build/Test
References: <6e47b64f0608030157p527d032en37dd11d9f92df4e3@mail.gmail.com>
 <44D1BCE1.6030007@gmail.com>
 <4d0b24970608030225i2317d2afrbef6dadc28bc8ffe@mail.gmail.com>
 <44D1C82A.8070805@gmail.com> <44D1DB2D.7030805@pobox.com>
 <4d0b24970608030636j59ba2bedn177da1823d5210cc@mail.gmail.com>
 <200608031409.k73E9LP9032049@d06av02.portsmouth.uk.ibm.com>
In-Reply-To: <200608031409.k73E9LP9032049@d06av02.portsmouth.uk.ibm.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit


Mark Hindess wrote:
> On 3 August 2006 at 21:36, "Andrew Zhang" <zhanghuangzhu@gmail.com> wrote:
>> On 8/3/06, Geir Magnusson Jr <geir@pobox.com> wrote:
>>> Paulex Yang wrote:
>>>> Andrew Zhang wrote:
>>>>> The failure is caused by Harmony-1040.
>>>>> The updated hostEquals(URL u1, URL u2) introduces a new bug:
>>>>> It returns true when u1.host = null and u2.host = "" while it should
>>>>> return
>>>>> false.
>>>>> Sorry for the mistake caused by my patch. I'll update a new patch to
>>>>> solve this problem.
> 
> Ideally we should also add a regression test to luni to catch this 
> problem.  The fact that the problem wasn't caught means the current
> luni don't have sufficient coverage so adding a test to luni would
> help improve this.

Whatever happened to "module + 1 dependency level" for testing?

> 
> I think we should always try to do this when a change in a module
> passes that modules tests but fails another modules tests (as long
> as it is reasonably practical to do so).
> 
> Regards,
>  Mark.
> 
>>>> Great, thank you to find this, Andrew. And don't need to say sorry, it's
>>>> me who takes responsibility on this and then owes the beer:-(.
>>> Out of curiosity, did either of you run the test suite on windows before
>>> offering the patch or committing it?  This isn't about "blame" or such,
>>> but about learning about how to make a better process....
>>
>> I did run luni and nio module test on both windows and linux, but security
>> module fails...
>>
>> It's a lesson for me. I'll pay more attention next time.
>>
>>
>>
>>> There's no harm done - it was found fast, and fixed fast, so kudos to
>>> you guys for working together on that.
>>>
>>> geir
> 
> 
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> 
> 
> 

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org