Hi: Sorry about the duplication of the code review request. I send out the first review earlier on Friday and it was over the quota limit due to the attachement. It needs moderator approval.
So I send out another review modifying the message without the attachment. I got already got code review feedback from Ginnie and Karen. I am going to send out another webrev with code review comments and additional test results. Thanks !!! ----- Original Message ----- From: [email protected] To: [email protected], [email protected] Sent: Saturday, June 16, 2012 2:35:09 AM GMT -08:00 US/Canada Pacific Subject: [caiman-discuss] Code review request for 7161787 Can I get two code reviewers for the following bug fix relating to L10N and this should be in s11u1_19: CR: http://monaco.sfbay/detail.jsf?cr=7161787 WPTG_TBT:Lint:CONCAT:Q&A204722-: Solaris 11.1 install_s11 T13Y issue in solaris_install_text_menu.po Web rev is here and it is based on suggestion from Takeshi Asano: https://cr.opensolaris.org/action/browse/caiman/md5645/7161787 Testing had been done this way: 1. I confirmed the solaris_install_text_menu.po got the suggested fixes: #: text-mode-menu.ksh:60 text-mode-menu.ksh:79 msgid "Terminal type (currently %s)" msgstr "" #: text-mode-menu.ksh:151 msgid "\"%s\" terminal type not supported." msgstr "" #: text-mode-menu.ksh:152 msgid "" "Supported terminal types can be found by using the Shell to list the " "contents of /usr/gnu/share/terminfo." msgstr "" 2. Currently, solaris_install_text_menu.mo had not been integrated yet. So I get a Japanese solaris_install_text_menu.mo from Takeshi-san and do a DC build with my install packages and the special solaris_install_text_menu.mo and test as follow In English, this is what happen when you use the new text-mode-menu: root@solaris:/var/svc/manifest# /usr/sbin/text-mode-menu Welcome to the Oracle Solaris installation menu 1 Install Oracle Solaris 2 Install Additional Drivers 3 Shell 4 Terminal type (currently sun-color) 5 Reboot Please enter a number [1]: 4 Indicate the type of terminal being used, such as: dtterm CDE terminal emulator xterm xterm vt100 DEC VT100 Enter terminal type [sun-color]: junk "junk" terminal type not supported. Supported terminal types can be found by using the Shell to list the contents of /usr/gnu/share/terminfo. Enter terminal type [sun-color]: In Japanese, this is a png of what you will see on the screen and it is working when I select 4. It is attached as attachment. root@solaris:~# text-mode-menu Oracle Solaris のインストールメニューへようこそ 1 Oracle Solaris のインストール 2 追加ドライバのインストール 3 シェル 4 端末のタイプ (現在 sun-color) 5 リブート 番号を入力してください[1]: 4 Indicate the type of terminal being used, such as: dtterm CDE terminal emulator xterm xterm vt100 DEC VT100 Enter terminal type [sun-color]: junk 端末タイプ junk はサポートされていません。 サポートされている端末タイプを確認するには、シェルを使用して /usr/gnu/share/terminfo の内容を一覧表示します。 Enter terminal type [sun-color]: _______________________________________________ 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
