Hi Ethan, I have just two nits - please see below.
Thank you, Jan installadm-common.sh -------------------- line 91 - we could perhaps use '\t' for tabs to compact that line and make use of tabs more explicit. 1147 - Would it be sufficient to use just '>' instead of '>|' ? You didn't introduce that code, so I'll let you decide if you want to change it or not. On 02/ 4/11 03:37 AM, Ethan Quach wrote:
Can I get a review for these two bugs. webrev: --------- http://cr.opensolaris.org/~equach/webrev.7015124.7016816/ bugIDs: ---------- http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7015124 http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7016816 thanks, -ethan _______________________________________________ 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

