LGTM.

-Drew

On 2/25/12 4:14 PM, Mary Ding wrote:
Hi:


I will like to ask for a code review for:

7143246 - pylint errors in solaris_install/transfer

http://monaco.us.oracle.com/detail.jsf?cr=7143246

Webrev:

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

The following pylints errors are fixed:


1.  W0611 - Unused import
2.  W0403 -  Relative import
3.  W0311 - Bad indentation


Testing:

1.  I ran the slim_install_units and there is no failure.

The test results are here:

/net/indiana-build/export/home/ding/7143246/slim_source/test_results

2. I used my packages to update x86 and sparc machines to do DC builds. The DC builds were done with all 5 images with my packages.

The isos were tested as follow:

X86 Live CD install - Pass
X86 TI Install - Pass
X86 AI install - Pass
Sparc TI Install - Pass
Sparc AI Install - Pass


_______________________________________________
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