Darren J Moffat wrote:
> Evan Layton wrote:
>> I need to get a quick re-review of the changes for the following bugs:
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=364
> 
> The change for this fix looks fine to me.  It is adding a specific entry 
> to /etc/auto_home for the user that was created during install.  I don't 
> see any need to run an ARC case for that, if it had been the generic 
> entry that would have been different.
> 
> 2350 +            fp.write(login + '     localhost:/export/home/&' + '\n')
> 
> Can't a '\t' be used instead of literal white space ?
> 

Yes it can and should be '\t'. I've made this change.

Thanks for looking at the changes!

-evan

Reply via email to