Karen:

I will ask Takeshi-san about this and get back to you.

On 05/22/12 04:51 PM, Karen Tung wrote:
Hi Mary,

I have a question for line 981.
In the original strings, there are multiple empty spaces preceding "OCM Hub URL".
I assume those are used for lining up with the string defined in
ASR_HUB_LABEL.  I see that those spaces
are also included inside the _(). Do you know whether that's OK? I am wondering whether the spacing will get messed up if the spaces are included, and the translated
string of "OCM Hub URL" is longer...

Thanks,

--Karen

On 05/22/12 15:09, Mary Ding wrote:
Hi:

Can I get a code review for the following bug fix for usr/src/cmd/system-config/support.py

CR: http://monaco.sfbay/detail.jsf?cr=7170341 several labels of text-install/sysconfig support screens are not localizable


Web rev is here and is suggested by Takeshi-san:

https://cr.opensolaris.org/action/browse/caiman/md5645/7170341

Testing is done this way:

1. Ran slim_install_unit tests for both sparc and x86. There were no regression.

x86 results pointer:

/net/tcu27-01/export/home/ding/7170341/slim_source/usr/src/tools/tests/test_result

sparc results pointer:

/net/osol-bldsp/datapool/ding/7170341/slim_source/usr/src/tools/tests/test_result

2.  Ran pep8 and pylint and there were no issues.

3.  I use DC to build TI images for sparc and x86.

x86 - TI installs works
sparc - TI installs works


_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to