Re: [docs] [PATCH] ref-manual/tasks.rst: Make do_clean description more explicit

"Antonin Godard" <[email protected]>
Newsgroups org.yoctoproject.lists.docs
Message-ID <[email protected]>
Hi,

On Mon Aug 10, 2026 at 2:06 AM CEST, anis via lists.yoctoproject.org wrote:
> The current do_clean task description is generic and does not
> specify which files will be removed.
>
> This branch adds details about the files that are removed by

This branch? I guess you mean this patch / this commit?

> explicitly listing the affected subdirectories.
>
> Signed-off-by: anis <[email protected]>
> ---
>  documentation/ref-manual/tasks.rst | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/documentation/ref-manual/tasks.rst b/documentation/ref-manual/tasks.rst
> index 3517d0b91..28de10d56 100644
> --- a/documentation/ref-manual/tasks.rst
> +++ b/documentation/ref-manual/tasks.rst
> @@ -17,7 +17,7 @@ you can run:
>     $ bitbake -c listtasks recipename
>     do_build                              Default task for a recipe - depends on all other normal tasks required to 'build' a recipe
>     do_checkuri                           Validates the SRC_URI value
> -   do_clean                              Removes all output files for a target
> +   do_clean                              Removes all output files for a target (``${WORKDIR}``, sstate manifests and stamps)

Since this is an excerpt from the listtasks task, this output should reflect
what's shown in reality. Maybe you should suggest this change to OE-Core to
change the "doc" flag for the task directly?

>     do_cleanall                           Removes all output files, shared state cache, and downloaded source files for a target
>     do_cleansstate                        Removes all output files and shared state cache for a target
>     do_compile                            Compiles the source in the compilation directory
> @@ -501,9 +501,10 @@ You can run this task using BitBake as follows::
>  
>     $ bitbake -c clean recipe
>  
> -Running this task does not remove the
> -:ref:`sstate <overview-manual/concepts:shared state cache>` cache files.
> -Consequently, if no changes have been made and the recipe is rebuilt
> +Running this task does remove the ``${``\ :term:`WORKDIR`\ ``}``, 

"does remove" -> "removes" feels better here

Suggestion:

"the ``${``\ :term:`WORKDIR`\ ``}``" -> "the working directory of the recipe
(represented by its :term:`WORKDIR` value)"

> +the sstate manifests and the stamps.

:ref:`stamps <overview-manual/concepts:Stamp Files and the Rerunning of Tasks>`

Tried to find a reference for sstate manifest files but couldn't find any :/

> +However, it does not remove the :ref:`sstate <overview-manual/concepts:shared state cache>`
> +cache files. Consequently, if no changes have been made and the recipe is rebuilt
>  after cleaning, output files are simply restored from the sstate cache.
>  If you want to remove the sstate cache files for the recipe, you need to
>  use the :ref:`ref-tasks-cleansstate` task instead

Thanks!
Antonin
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.