Dave Miner wrote: > Evan Layton wrote: >> I need to get a code review for the following simple fix: >> >> 5221 Improper message when creating a BE if no grub/menu.lst file >> http://defect.opensolaris.org/bz/show_bug.cgi?id=5221 >> >> Webrev: >> http://cr.opensolaris.org/~evanl/5221/ >> > > No issues with the revised code, per se, but it raises a related > question: how should a user recover from a situation where the menu file > has been deleted or otherwise mangled? Should beadm (or bootadm) have a > subcommand to regenerate a menu from an existing pool? > > Dave >
That's not only feasible it's a great idea (I wish I'd thought of it. ;-) ) I'm adding that and will resend the code review when I'm finished. Thanks! -evan
