Seems like that . Could you try generating a fresh patch rebased against the 
latest master code and attach it here ?  Otherwise, the changes look pretty 
minor ..I can add them manually .

Thanks,
Pranav

From: Channy Yun [mailto:[email protected]] On Behalf Of Channy Yun
Sent: Thursday, February 07, 2013 12:55 PM
To: Jessica Wang; Pranav Saxena
Cc: cloudstack; Channy Yun
Subject: Re: Review Request: CLOUDSTACK-1184 Localization - Add Korean label to 
all locales

This is an automatically generated e-mail. To reply, visit: 
https://reviews.apache.org/r/9359/



On February 7th, 2013, 7:06 a.m., Pranav Saxena wrote:

Hi Channy ,



Could you please check your patch once again ? It's not applying due to this-



error: patch failed: 
client/WEB-INF/classes/resources/messages_ru_RU.properties:1114

error: client/WEB-INF/classes/resources/messages_ru_RU.properties: patch does 
not apply



Thanks !

Hmm.. It seems good to me. Is it collision with wrong revision number?


- Channy


On February 7th, 2013, 6:53 a.m., Channy Yun wrote:
Review request for cloudstack, Pranav Saxena and Jessica Wang.
By Channy Yun.

Updated Feb. 7, 2013, 6:53 a.m.

Description

Korean translation was added, but it must be affected to all locales including 
messages.properties

Bugs: CLOUDSTACK-1184
Diffs

 *   client/WEB-INF/classes/resources/messages.properties (bb845d8)
 *   client/WEB-INF/classes/resources/messages_fr_FR.properties (51bf95f)
 *   client/WEB-INF/classes/resources/messages_ja.properties (8397a8d)
 *   client/WEB-INF/classes/resources/messages_ko_KR.properties (dc65d70)
 *   client/WEB-INF/classes/resources/messages_pt_BR.properties (23ba5b9)
 *   client/WEB-INF/classes/resources/messages_ru_RU.properties (024b552)
 *   client/WEB-INF/classes/resources/messages_zh_CN.properties (2e2e725)

View Diff<https://reviews.apache.org/r/9359/diff/>


Reply via email to