public inbox for docs@lists.yoctoproject.org
 help / color / mirror / Atom feed
* [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation
@ 2025-04-29 18:12 Enrico Jörns
  2025-04-30 10:21 ` [docs] " Quentin Schulz
  2025-05-21 16:42 ` Antonin Godard
  0 siblings, 2 replies; 3+ messages in thread
From: Enrico Jörns @ 2025-04-29 18:12 UTC (permalink / raw)
  To: docs; +Cc: yocto

* Since scarthgap [1], the default will be overridden when using
  'systemd' as INIT_MANAGER. Reflect this in the documentation.
* The distro configuration is probably the better place for
  customization, thus at least mention this together with the
  local.conf.
* While at it, drop the probably redundant description on how to
  override weak default variables.
  The example on how to set "/root" is still given indirectly, anyway.

[1] ebafe463 ("systemd: upgrade to 255.1")

Signed-off-by: Enrico Jörns <ejo@pengutronix.de>
---
 documentation/ref-manual/variables.rst | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index d17f81036..4527d15e4 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -7761,17 +7761,12 @@ system and gives an overview of their function and contents.
          prefer to have a read-only root filesystem and prefer to keep
          writeable data in one place.
 
-      You can override the default by setting the variable in any layer or
-      in the ``local.conf`` file. Because the default is set using a "weak"
-      assignment (i.e. "??="), you can use either of the following forms to
-      define your override::
+      When setting ``INIT_MANAGER = systemd``, the default will be set to::
 
-         ROOT_HOME = "/root"
          ROOT_HOME ?= "/root"
 
-      These
-      override examples use ``/root``, which is probably the most commonly
-      used override.
+      You can also override the default by setting the variable in your distro
+      configuration or in the ``local.conf`` file.
 
    :term:`ROOTFS`
       Indicates a filesystem image to include as the root filesystem.
-- 
2.39.5



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [docs] [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation
  2025-04-29 18:12 [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation Enrico Jörns
@ 2025-04-30 10:21 ` Quentin Schulz
  2025-05-21 16:42 ` Antonin Godard
  1 sibling, 0 replies; 3+ messages in thread
From: Quentin Schulz @ 2025-04-30 10:21 UTC (permalink / raw)
  To: ejo, docs; +Cc: yocto

Hi Enrico,

On 4/29/25 8:12 PM, Enrico Jörns via lists.yoctoproject.org wrote:
> * Since scarthgap [1], the default will be overridden when using
>    'systemd' as INIT_MANAGER. Reflect this in the documentation.
> * The distro configuration is probably the better place for
>    customization, thus at least mention this together with the
>    local.conf.
> * While at it, drop the probably redundant description on how to
>    override weak default variables.
>    The example on how to set "/root" is still given indirectly, anyway.
> 
> [1] ebafe463 ("systemd: upgrade to 255.1")
> 
> Signed-off-by: Enrico Jörns <ejo@pengutronix.de>

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

Thanks!
Quentin


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation
  2025-04-29 18:12 [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation Enrico Jörns
  2025-04-30 10:21 ` [docs] " Quentin Schulz
@ 2025-05-21 16:42 ` Antonin Godard
  1 sibling, 0 replies; 3+ messages in thread
From: Antonin Godard @ 2025-05-21 16:42 UTC (permalink / raw)
  To: docs, Enrico Jörns; +Cc: Thomas Petazzoni, Antonin Godard, yocto


On Tue, 29 Apr 2025 20:12:55 +0200, Enrico Jörns wrote:
> * Since scarthgap [1], the default will be overridden when using
>   'systemd' as INIT_MANAGER. Reflect this in the documentation.
> * The distro configuration is probably the better place for
>   customization, thus at least mention this together with the
>   local.conf.
> * While at it, drop the probably redundant description on how to
>   override weak default variables.
>   The example on how to set "/root" is still given indirectly, anyway.
> 
> [...]

Applied, thanks!

[1/1] ref-manual/variables.rst: update ROOT_HOME documentation
      commit: 50e92009d309fc4ae406174feb8f6578142748cc

Best regards,
-- 
Antonin Godard <antonin.godard@bootlin.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-05-21 16:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-29 18:12 [PATCH] ref-manual/variables.rst: update ROOT_HOME documentation Enrico Jörns
2025-04-30 10:21 ` [docs] " Quentin Schulz
2025-05-21 16:42 ` Antonin Godard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox