On 03/25/11 11:51 PM, Ethan Quach wrote:
Can I get a review for the following bugs:


BugIDs:
----------
http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7029179
http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7027956
http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7024643

Webrev:
----------
http://cr.opensolaris.org/~equach/webrev.7029179.7027956.7024643/


check-server-setup.sh, 143: s/broadcasted/advertised/

installadm-common.sh, 1072: s/form/from/
Also, it's a little troubling that there isn't any error checking of the result of the umount or mount operations here, but I guess you haven't made it any worse and ISIM should be handling this differently anyway.

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

Reply via email to