diff mbox series

[5/6] overview-manual: add UNPACKDIR in the directory description

Message ID 20250218-overview-figures-v1-5-75d23b5e7a88@bootlin.com
State New
Headers show
Series Updates around the UNPACKDIR variable | expand

Commit Message

Antonin Godard Feb. 18, 2025, 9:12 a.m. UTC
Mention that UNPACKDIR is used as a location to unpack the source code,
and that S is the final location of the source code. This is
deliberately vague, because as there are multiple instances of how these
directories can be defined and used.

The proper explanation of how the UNPACKDIR and S directories interact
is left to the reference manual, under the UNPACKDIR variable
description.

Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
---
 documentation/overview-manual/concepts.rst | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Quentin Schulz Feb. 20, 2025, 4:22 p.m. UTC | #1
Hi Antonin,

On 2/18/25 10:12 AM, Antonin Godard via lists.yoctoproject.org wrote:
> Mention that UNPACKDIR is used as a location to unpack the source code,
> and that S is the final location of the source code. This is
> deliberately vague, because as there are multiple instances of how these
> directories can be defined and used.
> 
> The proper explanation of how the UNPACKDIR and S directories interact
> is left to the reference manual, under the UNPACKDIR variable
> description.
> 

PR is also removed in this commit, maybe a separate one to explain 
what's happened to it would be nice? Or at the very least in the commit 
log so it doesn't look like a mistake?

Looks ok to me otherwise,

Cheers,
Quentin
diff mbox series

Patch

diff --git a/documentation/overview-manual/concepts.rst b/documentation/overview-manual/concepts.rst
index f0e2eb224908c149aaeada209af99e1477ea1cdd..e4d806f98ff2c8a9aca1b7c22ba0e0940a98acbc 100644
--- a/documentation/overview-manual/concepts.rst
+++ b/documentation/overview-manual/concepts.rst
@@ -704,10 +704,10 @@  a defined structure. For additional general information on the
 the Yocto Project Reference Manual.
 
 Each recipe has an area in the :term:`Build Directory` where the unpacked
-source code resides. The :term:`S` variable points to this area for a recipe's
-unpacked source code. The name of that directory for any given recipe is
-defined from several different variables. The preceding figure and the
-following list describe the :term:`Build Directory`'s hierarchy:
+source code resides. The :term:`UNPACKDIR` variable points to this area for a
+recipe's unpacked source code, and has the default ``sources-unpack`` name. The
+preceding figure and the following list describe the :term:`Build Directory`'s
+hierarchy:
 
 -  :term:`TMPDIR`: The base directory
    where the OpenEmbedded build system performs all its work during the
@@ -736,11 +736,11 @@  following list describe the :term:`Build Directory`'s hierarchy:
    -  :term:`PV`: The version of the
       recipe used to build the package.
 
-   -  :term:`PR`: The revision of the
-      recipe used to build the package.
+-  :term:`UNPACKDIR`: Contains the unpacked source files for a given recipe.
+
+-  :term:`S`: Contains the final location of the source code.
 
--  :term:`S`: Contains the unpacked source
-   files for a given recipe.
+   The default value for :term:`BP` is ``${BPN}-${PV}`` where:
 
    -  :term:`BPN`: The name of the recipe
       used to build the package. The :term:`BPN` variable is a version of