Mary - Looks good except for 1 change missing from cpio.py line 374. Not sure if it's the webrev or if it's my browser or if it's really not there.
Martin On 3/9/2011 1:00 PM, Mary Ding wrote:
Hi: Can I get a code review request for this: 7023927 - Need fix to take care of pep8audit tests failure for solaris_install/transfer This is the bugid detail: http://monaco.sfbay.sun.com/detail.jsf?cr=7023927 This is the web rev location: http://cr.opensolaris.org/~md5645/7023927/ For ips.py: line 284 and 709 were fixed with suggestions from Keith Mitchell line 451 is left alone to make the code more readable I had run install_unit_test and slim_code_cleanliness. I also use DC to build sparc and x86 test images and install them. There are no problems.
_______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
