Hi Sarah and Evan. On 10/13/09 06:44, Sarah Jelinek wrote: > Hi Evan, > > Looks ok. I do have a question: > > Why do we have this check: >> if ((self.LIVECD_INSTALL) or (self.AUTO_INSTALL)): >> 365 self.AUTOHOME = BASEDIR + AUTOHOME >> 366 else: >> 367 self.AUTOHOME = AUTOHOME >> 368 > > In ict.py? > > And, what happens when we add the text installer product? Don't we > have to modify this to include a check for that? Text-install image does introduce a third type here. I'll address this as part of the text-installer modification (part of the text-mode-menu work I've got for Driver Update).
Thanks, Jack > > thanks, > sarah > **** > > Evan Layton wrote: >> I need a couple of reviews for the fix to bug 364. This fix sets >> the users home directory to /home/<user_name> instead of >> /export/home/<user_name>. It then adds the "* >> localhost:/export/home/&" >> line to /etc/auto_home so that /home/<user_name> is auto-mounted on >> /export/home/<user_name>. >> >> The bug: >> http://defect.opensolaris.org/bz/show_bug.cgi?id=364 >> >> The webrev: >> http://cr.opensolaris.org/~evanl/364/ >> >> This ICT was tested from the command line as well as from >> an 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