Request for approval: 7094176: (tz) Incorrect TimeZone display name when DST not applicable / disabled

Deven You youdwei at linux.vnet.ibm.com
Wed Oct 31 23:59:06 PDT 2012


Thank Neil for committed this patch. And also thank Sean and Masayoshi 
for your review.

On 11/01/2012 01:00 AM, Neil Richards wrote:
> Both changesets now pushed to jdk7u-dev/jdk [1] [2].
>
> Regards,
> Neil
>
> [1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/87abc0c4804d
> [2] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/5341e088e9a3
>
> On Wed, 2012-10-31 at 11:00 +0000, Seán Coffey wrote:
>> Good point Masayoshi.
>>
>> Deven, to keep in sync with jdk8 you might consider importing the
>> 7172177 changeset also:
>> http://hg.openjdk.java.net/jdk8/tl/jdk/rev/60033ab79213
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7172177
>>
>> Consider that approved for jdk7u-dev push.
>>
>> regards,
>> Sean.
>>
>>
>> On 31/10/12 10:02, Masayoshi Okutsu wrote:
>>> Hi Deven,
>>>
>>> The fix itself looks good, but I believe the test was removed in JDK 8
>>> later.
>>>
>>> Thanks,
>>> Masayoshi
>>>
>>> On 10/31/2012 4:54 PM, Deven You wrote:
>>>> Hi All,
>>>>
>>>> This is  a request for approval to push the fix of 7094176 from jdk8
>>>> to jdk7u, could anyone take a look?
>>>>
>>>> Link to the bug:
>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7094176
>>>>
>>>> Changeset in jdk8:
>>>> http://hg.openjdk.java.net/jdk8/tl/jdk/rev/71cf74329a9e
>>>>
>>>> Webrev for jdk7u:
>>>> http://cr.openjdk.java.net/~youdwei/ojdk-557/webrev/
>>>> <http://cr.openjdk.java.net/%7Eyoudwei/ojdk-557/webrev/>
>>>>
>>>> The webrev for jdk7u is the same as of jdk8.
>>>>
>>>> Review thread for jdk8 starts at:
>>>>
>>>> http://mail.openjdk.java.net/pipermail/i18n-dev/2012-May/000613.html
>>>>
>>>> Thanks a lot!
>>>>
>




More information about the jdk7u-dev mailing list