Hi Dave. Thanks for taking a look.
On 02/10/10 11:39, Dave Miner wrote: > On 02/ 9/10 09:24 PM, Jack Schwartz wrote: >> Hi everyone. >> >> Here is the code review for AI changes to incorporate Driver Update. >> >> Driver Update will allow the installer to install packages of needed >> drivers in its own boot environment before performing the installation, >> and then install those same packages on the target. Packages may be >> explicitly specified in the manifest. They can also be found via search, >> also initiated via the manifest. >> >> http://cr.opensolaris.org/~schwartz/100209.1/webrev/index.html >> > > A couple of things that come to mind based on a quick skim: > > Comment in auto-install/default.xml for add_drivers is much too long, > it's mostly man-page fodder. One tag shouldn't add 30% to the size of > the file. OK. I'll trim it down. > > Do the changes in ai_sparc_image.xml and ai_x86_image.xml resolve the > issue in 14054? I think so. I need to update the webrev with that bug ID. I believe 14054 is part of 13109. 13109 calls out two issues: making /var/pkg/state RW (14054) and the need to increase the size of the bootroot. aside: I don't think 14054 is a dup of 13109, as 14054 is under our control but the size issue of 13109 is an issue for the IPS team. Thanks, Jack > > Dave > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
