[8] Request for review: 6192407: s10_70, ko, s1/dvd, minor misspelling under "Select Software Localizations" window

2013-10-29 Thread Michael Fang
Hello, Please help to review the changes for the following CR: https://bugs.openjdk.java.net/browse/JDK-6192407 The webrev is available here: http://cr.openjdk.java.net/~mfang/6192407/ thanks, -michael

Re:

2013-10-29 Thread Naoto Sato
Looks good to me. Naoto On 10/29/13, 2:32 PM, Michael Fang wrote: Hello, Please help to review the changes (1 line fix) for the following CR: https://bugs.openjdk.java.net/browse/JDK-8025521 The webrev is available here: http://cr.openjdk.java.net/~mfang/8025521/ The fix has been tested in a

i18n-dev@openjdk.java.net

2013-10-29 Thread Michael Fang
Hello, Please help to review the changes (1 line fix) for the following CR: https://bugs.openjdk.java.net/browse/JDK-8025521 The webrev is available here: http://cr.openjdk.java.net/~mfang/8025521/ The fix has been tested in a private build. thanks, -michael

Re: [8] Request for review: 8008647: [es] minor cosmetic issues in translated java command line outputs

2013-10-29 Thread Naoto Sato
Looks good to me. Naoto On 10/29/13, 2:08 PM, Michael Fang wrote: Hello, Please help to review the changes (1 line fix) for the following CR: https://bugs.openjdk.java.net/browse/JDK-8008647 The webrev is available here: http://cr.openjdk.java.net/~mfang/8008647/ The webrev doesn't show anyt

[8] Request for review: 8008647: [es] minor cosmetic issues in translated java command line outputs

2013-10-29 Thread Michael Fang
Hello, Please help to review the changes (1 line fix) for the following CR: https://bugs.openjdk.java.net/browse/JDK-8008647 The webrev is available here: http://cr.openjdk.java.net/~mfang/8008647/ The webrev doesn't show anything. The change is only adding a space on usage between "-x" and "e

Re: [jdk8]: exception in test Bug6317929 on a French platform.

2013-10-29 Thread Aleksej Efimov
Francis, Actually the test is updated in tl forest: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d34c5e860d5f But its still not merged to jdk8 forest. So, you can wait for a merge, or you can clone the tl forest and get the updated test. Aleksej On 29.10.2013 22:41, Francis ANDRE wrote: Hi Ale

Re: [jdk8]: exception in test Bug6317929 on a French platform.

2013-10-29 Thread Francis ANDRE
Hi Aleksej I run this command FrancisANDRE@idefix /cygdrive/Z/JDK/jdk8/jdk $ hg incoming comparing with http://hg.openjdk.java.net/jdk8/jdk8/jdk searching for changes no changes found and no recent changes were found... So, how do I get the latest version? Francis Le 29/10/2013 20:22, Alekse

Re: [jdk8]: exception in test Bug6317929 on a French platform.

2013-10-29 Thread Aleksej Efimov
Hello Francis, The following test was modified [1] to address a time zone names changes in JDK [2]. As I can see, you have an old version of this test. And just FYI - this test will be removed in future. Aleksej [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-October/022452.ht

[jdk8]: exception in test Bug6317929 on a French platform.

2013-10-29 Thread Francis ANDRE
Hi I got this exception running the Bug6317929 test on a French WXP/Cygwin/VS2010 platform with jdk8 which I modified locally to print the getDispalyName. So I am a little bit puzzled... What should be the proper result? Currie.getDisplayName=Eastern Normalzeit (Neusüdwales) Exception in thr

Re: [8] Request for review: 8026115: [zh_CN] inproper translation in output of jarsigner command

2013-10-29 Thread Michael Fang
Thanks Masayoshi for the review. I will work with translator to refix this bug. thanks, -michael On 13年10月28日 11:03 下午, Masayoshi Okutsu wrote: Should "[options]" and "alias..." in the second one be translated as well? {"Usage.jarsigner.options.jar.file.alias", -"用法

Re: [8] Request for review: 8025646: [pt_BR] overtranslation of option in java command line output

2013-10-29 Thread Naoto Sato
Looks good to me. Naoto On 10/28/13, 10:47 PM, Michael Fang wrote: Hello, Please help to review the changes (1 line fix) for the following CR: https://bugs.openjdk.java.net/browse/JDK-8025646 The webrev is available here: http://cr.openjdk.java.net/~mfang/8025646/ The change is in: FROM -ve

Re: Request for approval for 8025255: (tz) Support tzdata2013g

2013-10-29 Thread Seán Coffey
Approved for jdk7u-dev. I can push this changeset for you. regards, Sean. On 29/10/2013 11:04, Aleksej Efimov wrote: Hi, Another additional information for records: The test failure bug related to tzdata2013g update was approved for jdk8 [1]. The changeset [2] contains both fixes for 8026772

Re: RFR: 8026772: test/sun/util/resources/TimeZone/Bug6317929.java failing

2013-10-29 Thread Aleksej Efimov
Alan, I want to keep this test until I'll came up with new regression test . And I might will come up with some idea on this test modifications instead of deletion. So thats why I put it to ProblemsList.txt instead of deletion - some sort put it on hold with no impact to JDK 8 release process.

Re: RFR: 8026772: test/sun/util/resources/TimeZone/Bug6317929.java failing

2013-10-29 Thread Alan Bateman
On 24/10/2013 14:32, Aleksej Efimov wrote: Alan, Masayoshi, Michael, If you agree with the proposed changes then we can proceed with closing this bug. Can I ask you to sponsorship this change in such case? The hg patch located here: http://cr.openjdk.java.net/~aefimov/8026772/8026772_jdk8.patc

Re: Request for approval for 8025255: (tz) Support tzdata2013g

2013-10-29 Thread Aleksej Efimov
Hi, Another additional information for records: The test failure bug related to tzdata2013g update was approved for jdk8 [1]. The changeset [2] contains both fixes for 8026772 and 8025255. [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-October/022452.html [2] http://cr.openjdk