[2/2] manuals: SYSTEMD_SERVICE overrides depend on SYSTEMD_PACKAGES

Message ID 20220603163521.146659-2-ptsneves@gmail.com
State Accepted, archived
Delegated to: Armin Kuster
Headers show
Series [1/2] manuals: SYSTEMD_SERVICE allows multiple services | expand

Commit Message

Paulo Neves June 3, 2022, 4:35 p.m. UTC
Add detail that package overrides are only available if
specified in SYSTEMD_PACKAGES.
---
 documentation/ref-manual/variables.rst | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Michael Opdenacker June 6, 2022, 4:37 p.m. UTC | #1
Hi again Paulo,

On 6/3/22 18:35, Paulo Neves wrote:
> Add detail that package overrides are only available if
> specified in SYSTEMD_PACKAGES.
> ---
>  documentation/ref-manual/variables.rst | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
> index ed52c3508..6e2919474 100644
> --- a/documentation/ref-manual/variables.rst
> +++ b/documentation/ref-manual/variables.rst
> @@ -7664,6 +7664,10 @@ system and gives an overview of their function and contents.
>  
>           SYSTEMD_SERVICE:${PN} = "connman.service"
>  
> +      The package overrides that can be specified are directly related to the value of
> +      term: SYSTEMD_PACKAGES. Overrides not included in term: SYSTEMD_PACKAGES
> +      will be silently ignored.


You should use instead:
:term:`SYSTEMD_PACKAGES`

Would you mind sending an update to docs@lists.yoctoproject.org with
your "Signed-off-by" information, please?

Thanks in advance
Michael.

Patch

diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index ed52c3508..6e2919474 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -7664,6 +7664,10 @@  system and gives an overview of their function and contents.
 
          SYSTEMD_SERVICE:${PN} = "connman.service"
 
+      The package overrides that can be specified are directly related to the value of
+      term: SYSTEMD_PACKAGES. Overrides not included in term: SYSTEMD_PACKAGES
+      will be silently ignored.
+
    :term:`SYSVINIT_ENABLED_GETTYS`
       When using
       :ref:`SysVinit <dev-manual/common-tasks:enabling system services>`,