installadm-common.sh: Are we at all concerned with the case of an image that might exist with the amd64 boot_archive a link to the 32 bit one? Is there any way of combining the two if's into one? Just seems a little awkward to check for basically the same thing in two different ways. Also seems like we could end up with an issue if x86.microroot and the 64 bit boot_archive both existed. Not that that is likely.
Jean Jan Damborsky wrote: > Hi Jean, Alok, > > could I please ask you to review fix for following bugs ? > > <http://defect.opensolaris.org/bz/show_bug.cgi?id=7966>7966 setting > 'livemode=text' is no-op for AI, should be removed from installadm(1M) > 8140 Fix for 6440 applied to x86 AI images would reduce x86 AI memory > consumption > > > Thank you very much, > Jan > > webrev: > http://cr.opensolaris.org/~dambi/bug-8140/ > > testing done: > please see attachment for tests carried out and results >
