Review request: Test bugs to add cygwin support

2011-08-08 Thread Mandy Chung
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

Re: Review request: Test bugs to add cygwin support

2011-08-08 Thread Xueming Shen
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

Re: Review request: Test bugs to add cygwin support

2011-08-08 Thread Mandy Chung
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

Re: Review request: Test bugs to add cygwin support

2011-08-08 Thread joe . darcy
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