[yocto] [yocto-docs][PATCH] kernel-dev: document the change detection of kernel feature files

[Bruce Ashfield]

On 2018-10-01 8:42 AM, Urs Fässler wrote:
> Recommend to add recipe-space features to SRC_URI so that changes to them
> are detected automatically.
> 
> Signed-off-by: Urs Fässler <urs.fassler at bbv.ch>
> Signed-off-by: Pascal Bach <pascal.bach at siemens.com>
> ---
>   documentation/kernel-dev/kernel-dev-common.xml | 7 +++++--
>   1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/documentation/kernel-dev/kernel-dev-common.xml b/documentation/kernel-dev/kernel-dev-common.xml
> index 83b02b1c1..6289ce8d4 100644
> --- a/documentation/kernel-dev/kernel-dev-common.xml
> +++ b/documentation/kernel-dev/kernel-dev-common.xml
> @@ -2623,6 +2623,9 @@
>               <filename>.scc</filename> file in the
>               <filename>SRC_URI</filename> statement to reference multiple kernel
>               features.
> +            Since BitBake only detects changes on files listed in
> +            <filename>SRC_URI</filename>, it is best to add all

This is not necessarily true. It would be better stated that if you
are modifying the .cfg files in the layer directly, you should list
them on the SRC_URI to have the changes picked up by bitbake.

It is not a matter of "best", so it should be stated that way.

> +            <filename>.cfg</filename> to <filename>SRC_URI</filename>.
>           </para>
>   
>           <para>
> @@ -2674,10 +2677,10 @@
>                       </para></listitem>
>                   <listitem><para>
>                       <emphasis>Add the Feature File to <filename>SRC_URI</filename>:</emphasis>
> -                    Add the <filename>.scc</filename> file to the
> +                    Add the <filename>.cfg</filename> and <filename>.scc</filename> file to the
>                       recipe's <filename>SRC_URI</filename> statement:
>                       <literallayout class='monospaced'>
> -     SRC_URI_append = " file://test.scc"
> +     SRC_URI_append = " file://test.cfg file://test.scc"

This is wrong. You should not have both on the SRC_URI

Bruce

>                       </literallayout>
>                       The leading space before the path is important as the
>                       path is appended to the existing path.
>