This is already fixed in pybootmgmt. --S
On Apr 2, 2012, at 7:35 AM, Sue Sohn <[email protected]> wrote: > Hi Niall, > > Quick question not about the changes you made, but about a comment I noticed > in the webrev (these are b14 line numbers): > > 820 # XXX Seth: Pybootmgmt ought to be creating this directory > tree > 821 # when doing a physical boot loader installation onto the > disk. > 822 # While it's being fixed, create the directories ourselves to > 823 # make legacy Grub installation work. > > Was just wondering about this and if a bug should be filed to track it. > > Sue > > On 04/ 1/12 08:50 PM, Niall Power wrote: >> Hi, >> >> Install PIT discovered a problem on SPARC that in some circumstances could >> caues boot loader >> installation to fail on SPARC. >> This is also a potential issue for X86 systems. Due to the severity I'd like >> to, if at all possible, >> get this fix into build 13. >> >> 7158052 - Installation fails due to installboot returned unexpected exit >> status 4 in s11u1_13. >> >> There are 2 webrevs: one for build 13 and one for build 14. >> >> The difference is that the b14 webrev is a subset of the b13 fix. The reason >> for that is that in >> build 14 we switched to using pybootmgmt to handle boot loader installation >> on both X86 AND SPARC, >> whereas upto build 13, we handled SPARC boot loader setup directly. >> >> For b13, the installboot(1M) needs to be passed the "-f" switch to force >> boot loader installation, >> and when using pybootmgmt for X86, commit_boot_config() needs to be passed >> the "force=True" arg val. >> >> For b14 we just need to carry forward the "force=True" arg val when invoking >> commit_boot_config() >> >> Webrevs: >> ------------ >> Build 13: >> http://jurassic.us.oracle.com/~npower/webrevs/webrev-force-bootblck-b13/ >> >> Build 14: >> http://jurassic.us.oracle.com/~npower/webrevs/webrev-force-bootblck-b14/ >> >> >> I've built build 13 DC SPARC and X86 ISO images incorporating the fix. Seth >> has confirmed that they >> fix the problem. >> He has also already reviewed the webrev and given this a +1 >> >> Thanks, >> Niall >> _______________________________________________ >> 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
