From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Young Subject: Re: [PATCH 1/5 branch usrmove v2] deal common part of etc passwd in 99base Date: Thu, 29 Dec 2011 18:00:14 +0800 Message-ID: <4EFC3A2E.6020606@redhat.com> References: <20111226062910.GA3141@darkstar.nay.redhat.com> <4EFBD882.2090707@redhat.com> <4EFC36CD.8040205@redhat.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: initramfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Cong Wang Cc: initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On 12/29/2011 05:48 PM, Cong Wang wrote: > On Thu, Dec 29, 2011 at 5:45 PM, Dave Young wrote: >> On 12/29/2011 05:27 PM, Cong Wang wrote: >>> The nfs module still uses "/" as home of root, while ssh module uses >>> "/root". With you patches, you changed to "/root" for both nfs and >>> ssh, which *maybe* a problem for nfs module since you didn't test it. >> >> >> I tested nfs boot as well, it works ok with $HOME=/root > > Ok, will change to "/root" too. > >> >> Also if you add both ssh-client and nfs, what's the result of /etc/passwd? > > If the line "root:" exists before, > > egrep -q '^root:' "$initdir/etc/passwd" 2>/dev/null || echo > 'root:x:0:0::/:/bin/sh' >> "$initdir/etc/passwd" > > egrep will succeed and echo will not be executed... Yes, but who knows which module will be added firstly, maybe there's other module do this in future. But if nfs also use /root as HOME it will be ok currently -- Thanks Dave