On 2/12/10 12:20 PM, Evan Layton wrote: > On 2/12/10 11:34 AM, Dermot McCluskey wrote: >> Evan, >> >> This looks file, but it seems (from reading the bug) that >> in previous releases, /boot/grub/menu.lst was overwritten >> with a stub file that referred to the "correct" menu.lst >> file in the root pool.
I think what you're referring to here is the comment in 6925996 about what the old installer did. We don't doing anything with the /boot/grub/menu.lst file anymore and only make changes to the /<pool>/boot/grub/menu.lst file from within the ICT's. -evan >> >> Is that not a better fix? ie might it confuse a user if >> /boot/grub/menu.lst contains valid entries, but isn't >> being used? >> > > This is being addressed by bug 6925996 > http://monaco.sfbay.sun.com/detail.jsf?cr=6925996 > > -evan > >> >> - Dermot >> >> >> >> >> On 02/12/10 18:16, Evan Layton wrote: >>> I need to get a couple of reviews for this very simple fix. It needs to >>> go into both branches and is a stopper bug. >>> >>> webrev: >>> http://cr.opensolaris.org/~evanl/14475/ >>> >>> Defect: >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=14475 >>> >>> Tested by building images and installing to make sure that >>> /boot/grub/menu.lst still existed after the install. >>> >>> Thanks! >>> -evan >>> _______________________________________________ >>> caiman-discuss mailing list >>> caiman-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
