Review request: Test bugs to add cygwin support

joe.darcy at oracle.com joe.darcy at oracle.com
Mon Aug 8 23:22:32 UTC 2011


Looks fine.

-Joe

On 8/8/2011 4:06 PM, Mandy Chung wrote:
> Sherman,
>
> Thanks.  Can you review one more fix:
>
> 7012365: TEST_BUG: test/java/nio/charset/spi/basic.sh can be run with 
> Cygwin
> Webrev at:
>   http://cr.openjdk.java.net/~mchung/7012365/webrev.00/
>
> Mandy
>
> On 8/8/11 3:16 PM, Xueming Shen wrote:
>> Looks good. Thanks for fixing them.
>>
>> On 08/08/2011 02:37 PM, Mandy Chung wrote:
>>> Simple fix on these tests to run on cygwin.
>>>
>>> 7036518: TEST_BUG: add cygwin support to
>>> test/java/nio/charset/coders/CheckSJISMappingProp.sh
>>> 7036519: TEST_BUG: add cygwin support to test/demo/zipfs/basic.sh
>>>
>>> Webrev at:
>>>       http://cr.openjdk.java.net/~mchung/7036518/webrev.00/
>>>
>>> Thanks
>>> Mandy
>



More information about the core-libs-dev mailing list