LGTM.
-Drew
On 3/16/12 9:55 AM, Niall Power wrote:
Hi,
Can I get a sanity check for this webrev for distro constructor.
This has already been reviewed as part of the overall project webrev
for UEFI support, but we have an opportunity to integrate a UEFI and
GRUB2 compatible distro-constructor before the actual delivery of UEFI
and GRUB2 in build 14.
So what I have done is extract a subset of the full webrev that
delivers just the distro constructor code changes. The separation is
quite clean and easily isolated from the rest of the code base. There
was no recoding necessary.
I'd like to ask for a quick sanity check however from folks who
previously reviewed the distro-constructor and boot checkpoint pieces
as part of my original webrev for UEFI.
I've tested this webrev to confirm that it
- builds a GRUB2 BIOS/UEFI based hybrid ISO image (confirmed that said
image boots)
- builds existing legacy GRUB based BIOS ISO images
- confirmed that existing installadm can import both the ISO and AI
pkg repo image for the legacy GRUB image
- confirmed that the usbcopy and usbgen function with both the legacy
GRUB and GRUB2 based ISO images and boot correctly for BIOS and UEFI
(for GRUB2)
Pointer to DC UEFI webrev:
http://jurassic.us.oracle.com/~npower/webrevs/webrev-dc-uefi-2012-03-16/
Pointer to overall slim_uefi webrev from which this sub-webrev is
taken from:
http://jurassic.us.oracle.com/~npower/webrevs/webrev-slim-uefi-2012-03-16/
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
