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: 202.81.18.152 is neither permitted nor
 denied by domain of paulex.yang@gmail.com)
Message-ID: <44BD9BCA.1060306@gmail.com>
Date: Wed, 19 Jul 2006 10:41:14 +0800
From: Paulex Yang <paulex.yang@gmail.com>
User-Agent: Thunderbird 1.5.0.4 (Windows/20060516)
MIME-Version: 1.0
To: harmony-dev@incubator.apache.org
Subject: Re: [build] status
References: <44BC99FA.1000103@gmail.com> <44BCEF56.70707@pobox.com>
 <44BD02E4.301@pobox.com> <44BD13EE.1000506@gmail.com>
 <44BD95D5.9090300@gmail.com> <44BD990D.7080900@pobox.com>
In-Reply-To: <44BD990D.7080900@pobox.com>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

Geir Magnusson Jr wrote:
> Then I guess we just fix the test.
>   
Agree, just think we should not "fix" the test by removing the assertion.
> geir
>
>
> Paulex Yang wrote:
>   
>> Tim Ellison wrote:
>>     
>>> Geir Magnusson Jr wrote:
>>>  
>>>       
>>>> Now, on my windows box I got a clean build, no failures....  hm.
>>>>     
>>>>         
>>> The test works on en_US locale and fails on en_UK.  I'm guessing that
>>> your machine is set up as en_US.
>>>
>>> Richard offered a patch that sets the locale to en_US for all
>>> MessageFormatTest-s, but I suggested that was not a suitable solution.
>>>
>>> For now I suggest we remove the assertion, since it is beyond the spec
>>> requirements for this type.
>>>   
>>>       
>> Tim,
>>
>> Should we also consider the principle of "follow the RI" here?  I think
>> it is a sample of unclear spec, and we should assert if our
>> implementation follows RI, i.e., the assert about return value of
>> toPattern() is necessary. The issue here is just that the testcase
>> assumes the return value is locale-independent, so I think Richard's
>> patch is OK. Further, from the test perspective, maybe(just maybe) test
>> case on only one locale is not enough to check Harmony's toPattern()
>> logic, tests on more different locale(especially the 'exceptional' case
>> like en_UK) are better.
>>
>> Also FYI, I tried the original test case with RI on en_UK locale, and it
>> failed exactly as Harmony.
>>     
>>> Regards,
>>> Tim
>>>
>>>
>>>  
>>>       
>>>> Geir Magnusson Jr wrote:
>>>>    
>>>>         
>>>>> This is nuts.  We need to chase down the commit that broke this, and
>>>>> reverse it.  We can't have a broken build persisting this long.
>>>>>
>>>>> geir
>>>>>
>>>>>
>>>>> Tim Ellison wrote:
>>>>>      
>>>>>           
>>>>>> (1) Linux build/tests are passing again, but for some reason the
>>>>>> 'BUILD SUCCESSFUL' note didn't go to the commit list?
>>>>>>
>>>>>>
>>>>>> (2) Windows build/test is still failing with:
>>>>>>
>>>>>> Wrong full date pattern expected:<...full...> but was:<...long...>
>>>>>>
>>>>>> junit.framework.ComparisonFailure: Wrong full date pattern
>>>>>> expected:<...full...> but was:<...long...> at
>>>>>> org.apache.harmony.text.tests.java.text.MessageFormatTest.test_applyPatternLjava_lang_String(MessageFormatTest.java:244)
>>>>>>
>>>>>> at
>>>>>> java.lang.reflect.AccessibleObject.invokeV(AccessibleObject.java:205)
>>>>>>
>>>>>>
>>>>>> Regards,
>>>>>> Tim
>>>>>>
>>>>>>         
>>>>>>             
>>>>> ---------------------------------------------------------------------
>>>>> 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
>>>>
>>>>
>>>>     
>>>>         
>>>   
>>>       
>>     
>
> ---------------------------------------------------------------------
> 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
>
>
>   


-- 
Paulex Yang
China Software Development Lab
IBM


---------------------------------------------------------------------
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