[prev in list] [next in list] [prev in thread] [next in thread] 

List:       initramfs
Subject:    Re: [PATCH] modules.d: introduce ssh-server module to provide service
From:       Pingfan Liu <piliu () redhat ! com>
Date:       2017-06-22 2:23:16
Message-ID: 1765900923.25427268.1498098196908.JavaMail.zimbra () redhat ! com
[Download RAW message or body]





----- Original Message -----
> From: "Harald Hoyer" <harald@redhat.com>
> To: "Liu ping fan" <kernelfans@gmail.com>, "Pingfan Liu" <piliu@redhat.com>
> Cc: initramfs@vger.kernel.org
> Sent: Wednesday, June 21, 2017 10:41:50 PM
> Subject: Re: [PATCH] modules.d: introduce ssh-server module to provide service
> 
> On 12.06.2017 07:47, Liu ping fan wrote:
> > Hi Harald, what is your opinion of such functionality?
> 
> yeah, nice, but some issues:
> 
> 1) "inst" should be used instead of "inst_simple" for executables and *.so
> 2) That is a long hardcoded list of dependencies. Are they all needed?
> 3) The module should probably be renamed to rhel-7-ssh-server or a similar
> name
> to reflect, that it is most likely only working on this distribution
> variant
> 4) Does it have to be shipped in upstream dracut? Looks like this can live
> totally
> fine in a project, which lives outside of dracut.
> 
Ah, I brought up this patch to ease the remote debugging on customer env. And usually \
the bug reporter is not familiar with detail, and it cost some time to communicate. \
With this patch, it is more easy to debug the initrd issue with capture-kernel. It is \
ok if you think it is better to keep this patch lives outside of dracut. I can ask \
the customer to install it by manual.



Thanks and regards,
Pingfan
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[prev in list] [next in list] [prev in thread] [next in thread]