Good afternoon!

Could I get a code review for two CRs:

7120436 <http://monaco.us.oracle.com/detail.jsf?cr=7120436> TI throw exception if try to delete extended partition and install in solaris2 partition 7164336 <http://monaco.us.oracle.com/detail.jsf?cr=7164336> new pep8 error in usr/src/cmd/text-install/partition_edit_screen.py after changeset 1655:4e0a5494f84

https://cr.opensolaris.org/action/browse/caiman/drewfish/7120436/webrev/

I tested 7120436 by creating a couple of logical partitions and verifying text-install doesn't crash in final validation when leaving the screen.

Note: F7 resets everything back to what's in the discovered tree. The logical partitions are not permanently removed.

pep8 and pylint comes back clean.

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

Reply via email to