On 3/2/10 9:17 AM, Susan Sohn wrote: > On 03/01/10 21:13, Evan Layton wrote: >> I need to get a review for bug 14962. >> >> Bug: >> http://defect.opensolaris.org/bz/show_bug.cgi?id=14962 >> >> Webrev: >> http://cr.opensolaris.org/~evanl/14962/ >> >> Some background: >> This bug resolves the issue related to installing in a logical >> partition where the first time a BE is activated the activate >> will fail due to running installgrub when it doesn't need to. >> >> This does not resolve the issues related to bug 14894. >> >> Thanks, >> -evan >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss > > Hi Evan, > > Couple of nits in ict.py: > 2273-2274 add a space after the # per pep8 > 2286 '/boot/grub/menu.lst :' -> '/boot/grub/menu.lst: '
Fixed. Thanks for looking at it! -evan