Re: [docs] [PATCH v2 19/56] kernel-dev/common.rst: remove obsolete poky repo references

[Quentin Schulz <quentin.schulz@cherry.de>]

Hi Antonin,

On 1/27/26 9:19 AM, Antonin Godard via lists.yoctoproject.org wrote:
> Remove references to the Poky repository, replace by OpenEmbedded-Core.
> 
> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> ---
>   documentation/kernel-dev/common.rst | 112 +++++++++++++++---------------------
>   1 file changed, 46 insertions(+), 66 deletions(-)
> 
> diff --git a/documentation/kernel-dev/common.rst b/documentation/kernel-dev/common.rst
> index 2142d31c0..a653d897c 100644
> --- a/documentation/kernel-dev/common.rst
> +++ b/documentation/kernel-dev/common.rst
> @@ -44,25 +44,15 @@ image and ready to make modifications as described in the
>   ":ref:`kernel-dev/common:using \`\`devtool\`\` to patch the kernel`"
>   section:
>   
> -#. *Initialize the BitBake Environment:*
> -   you need to initialize the BitBake build environment by sourcing
> -   the build environment script (i.e. :ref:`structure-core-script`)::
> -
> -      $ cd poky
> -      $ source oe-init-build-env
> -
> -   .. note::
> -
> -      The previous commands assume the
> -      :ref:`overview-manual/development-environment:yocto project source repositories`
> -      (i.e. ``poky``) have been cloned using Git and the local repository is named
> -      "poky".
> +#. *Initialize the Build Environment:* Follow the :doc:`/dev-manual/start`
> +   section of the Yocto Project Development Tasks Manual in order to have a
> +   ready-to-use shell that can execute ``devtool``.
>   
>   #. *Prepare Your local.conf File:* By default, the :term:`MACHINE` variable
>      is set to "qemux86-64", which is fine if you are building for the QEMU
>      emulator in 64-bit mode. However, if you are not, you need to set the
>      :term:`MACHINE` variable appropriately in your ``conf/local.conf`` file
> -   found in the :term:`Build Directory` (i.e.  ``poky/build`` in this example).
> +   found in the :term:`Build Directory` (i.e.  ``bitbake-builds/build`` in this example).
>   

I... just realized that bitbake-setup init actually creates an 
additional directory. It is bitbake-builds/poky-master/build actually. Sigh.

>      Also, since you are preparing to work on the kernel image, you need
>      to set the :term:`MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS` variable to include
> @@ -79,7 +69,7 @@ section:
>      patches created for the kernel image. You can use the
>      ``bitbake-layers create-layer`` command as follows::
>   
> -      $ cd poky/build
> +      $ cd bitbake-builds/build
>         $ bitbake-layers create-layer ../../meta-mylayer

I would suggest to change this path to ../layers/meta-mylayer such that 
it follows our recommendation of having all the layers in the same 
place. In a separate patch/series is fine.

This patch looks fine though,

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin