Mary,

Looks good to me also.

John

On 05/17/12 10:40 AM, Mary Ding wrote:
Darren:

Thanks.  I still need one more code reviewer for this fix.

On 05/17/12 09:20 AM, Darren Kenny wrote:
LGTM,

Thanks,

Darren.

On 17/05/2012 17:16, Mary Ding wrote:
Hi:

Can I have a code review for the following L10N message bug fix:

7169555<http://monaco.us.oracle.com/detail.jsf?cr=7169555>
several recently added help files are missing in install-message-files

Webrev is here and is provided by Takeshi Asano:

https://cr.opensolaris.org/action/browse/caiman/asano/7169555/

Testing is done as follow:

1.  I had done slim_source build and confirm the build works for both
sparc and x86.

2.  I checked the
publisher/install-localizable/pkg/consolidation%2Finstall%2Finstall-message-files
and confirmed the missing help files are in the install-message-files

Thanks !!!




_______________________________________________
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

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

Reply via email to