<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 13, 2019 at 9:59 PM Hongzhi, Song <<a href="mailto:hongzhi.song@windriver.com">hongzhi.song@windriver.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
On 8/13/19 8:27 PM, Bruce Ashfield wrote:<br>
><br>
><br>
> On Tue, Aug 13, 2019 at 1:35 AM Hongzhi.Song <br>
> <<a href="mailto:hongzhi.song@windriver.com" target="_blank">hongzhi.song@windriver.com</a> <mailto:<a href="mailto:hongzhi.song@windriver.com" target="_blank">hongzhi.song@windriver.com</a>>> wrote:<br>
><br>
>Â Â Â A new patch let kernel source Documentation/Kconfig in top Kconfig<br>
>Â Â Â So kernel-devsrc should include Documentation/ too.<br>
>Â Â Â Otherwise "make scripts" will fails.<br>
><br>
>Â Â Â patch:<br>
>Â Â Â commit b1663d7e3a7961fc45262fd68a89253f2803036c<br>
>Â Â Â Author: Mauro Carvalho Chehab <<a href="mailto:mchehab%2Bsamsung@kernel.org" target="_blank">mchehab+samsung@kernel.org</a><br>
>Â Â Â <mailto:<a href="mailto:mchehab%252Bsamsung@kernel.org" target="_blank">mchehab%2Bsamsung@kernel.org</a>>><br>
>Â Â Â Date:Â Â Tue Jun 4 09:26:27 2019 -0300<br>
><br>
>Â Â Â Â Â docs: Kbuild/Makefile: allow check for missing docs at build time<br>
><br>
>Â Â Â Â Â While this doesn't make sense for production Kernels, in order to<br>
>Â Â Â Â Â avoid regressions when documents are touched, let's add a<br>
>Â Â Â Â Â check target at the make file.<br>
><br>
>Â Â Â Â Â Signed-off-by: Mauro Carvalho Chehab<br>
>Â Â Â <<a href="mailto:mchehab%2Bsamsung@kernel.org" target="_blank">mchehab+samsung@kernel.org</a> <mailto:<a href="mailto:mchehab%252Bsamsung@kernel.org" target="_blank">mchehab%2Bsamsung@kernel.org</a>>><br>
>Â Â Â Â Â Signed-off-by: Jonathan Corbet <<a href="mailto:corbet@lwn.net" target="_blank">corbet@lwn.net</a><br>
>Â Â Â <mailto:<a href="mailto:corbet@lwn.net" target="_blank">corbet@lwn.net</a>>><br>
><br>
>Â Â Â Signed-off-by: Hongzhi.Song <<a href="mailto:hongzhi.song@windriver.com" target="_blank">hongzhi.song@windriver.com</a><br>
>Â Â Â <mailto:<a href="mailto:hongzhi.song@windriver.com" target="_blank">hongzhi.song@windriver.com</a>>><br>
>Â Â Â ---<br>
>Â Â Â Â meta/recipes-kernel/linux/<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">kernel-devsrc.bb</a><br>
>Â Â Â <<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">http://kernel-devsrc.bb</a>> | 2 +-<br>
>Â Â Â Â 1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
>Â Â Â diff --git a/meta/recipes-kernel/linux/<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">kernel-devsrc.bb</a><br>
>Â Â Â <<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">http://kernel-devsrc.bb</a>><br>
>Â Â Â b/meta/recipes-kernel/linux/<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">kernel-devsrc.bb</a> <<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">http://kernel-devsrc.bb</a>><br>
>Â Â Â index 5ec5929..a874e06 100644<br>
>Â Â Â --- a/meta/recipes-kernel/linux/<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">kernel-devsrc.bb</a><br>
>Â Â Â <<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">http://kernel-devsrc.bb</a>><br>
>Â Â Â +++ b/meta/recipes-kernel/linux/<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">kernel-devsrc.bb</a><br>
>Â Â Â <<a href="http://kernel-devsrc.bb" rel="noreferrer" target="_blank">http://kernel-devsrc.bb</a>><br>
>Â Â Â @@ -65,7 +65,7 @@ do_install() {<br>
>Â Â Â Â Â Â )<br>
><br>
>Â Â Â Â Â Â # then drop all but the needed Makefiles/Kconfig files<br>
>Â Â Â -Â Â rm -rf $kerneldir/build/Documentation<br>
>Â Â Â +Â Â #rm -rf $kerneldir/build/Documentation<br>
><br>
><br>
> In the spirit of keeping kernel-devsrc as small as possible (I have <br>
> another patch pending if you really want the full kernel source), this <br>
> should only keep the Documentation/ files that are required to pass <br>
> the check, not keep all of Documentation.<br>
<br>
<br>
If you have a better patch, I am pleasure to accept it.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">???</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">This is where you'd typically do a v2 of the patch after getting a review of a change.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">But if you are refusing the feedback, then yes, I'll do a version of the patch myself rather than just blindly copying in all of the documentation. I'll submit it myself.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">RP/Ross, whoever is taking in patches, drop this version, and I'll do my own.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Cheers,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Bruce</div><br></div><div>Â </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
<br>
--Hongzhi<br>
<br>
<br>
> Bruce<br>
><br>
>Â Â Â Â Â Â rm -rf $kerneldir/build/scripts<br>
>Â Â Â Â Â Â rm -rf $kerneldir/build/include<br>
><br>
>Â Â Â -- <br>
>Â Â Â 2.8.1<br>
><br>
>Â Â Â -- <br>
>Â Â Â _______________________________________________<br>
>Â Â Â Openembedded-core mailing list<br>
>Â Â Â <a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
>Â Â Â <mailto:<a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a>><br>
>Â Â Â <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
><br>
><br>
><br>
> -- <br>
> - Thou shalt not follow the NULL pointer, for chaos and madness await <br>
> thee at its end<br>
> - "Use the force Harry" - Gandalf, Star Trek II<br>
><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end<br>- "Use the force Harry" - Gandalf, Star Trek II<br><br></div></div>