From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Young Subject: Re: [PATCH 1/4 branch usrmove] deal common part of etc passwd in 99base Date: Mon, 26 Dec 2011 13:42:40 +0800 Message-ID: <4EF80950.5040700@redhat.com> References: <20111223080420.GA12664@darkstar.nay.redhat.com> <4EF7D46E.4000109@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/26/2011 12:58 PM, Cong Wang wrote: > On Mon, Dec 26, 2011 at 9:57 AM, Dave Young wrote: >> On 12/25/2011 07:49 PM, Cong Wang wrote: >> >>> On Fri, 23 Dec 2011 at 08:04 GMT, Dave Young wrote: >>>> ssh module will need root user in /etc/passwd, so add root and nobody >>>> to /etc/passwd in 99base instead of nfs module >>> >>> Huh? Even for who don't use either ssh module or nfs module?? >>> >>> I think you just need to copy these two lines into your ssh module... >> >> >> Thanks for your review. >> >> in 90mdraid module there's words in its comments about passwd, so there >> might be more modules need passwd. > > But there are still more modules that don't need passwd, right? > >> >> Dealing with same thing in multi module is not good, not only duplicate >> code, but also they could cause conflict. Ie. nfs set root home dir to / >> but we need /root, if nfs module added after ssh-client the home dir of >> root will be /. > > If you don't want to duplicate the code, make it a function (maybe in > dracut-function?), > putting it in base module is confusing. Scratching head.. I'm feeling bad to add same user multi-times, Yes not all modules need passwd, but I still think it is a base thing. > -- > To unsubscribe from this list: send the line "unsubscribe initramfs" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks Dave