diff mbox series

bitbake: user-manual: inform about spaces in :remove

Message ID 20220929121135.259094-1-johan.korsnes@remarkable.no
State New
Headers show
Series bitbake: user-manual: inform about spaces in :remove | expand

Commit Message

Johan Korsnes Sept. 29, 2022, 12:11 p.m. UTC
Inform the reader that there should be no need for spaces in the value
when using removal override `:remove`.

Considering why spaces are used in the other override operators, it
might seem obvious that they aren't needed for the removal operator.
But, it seems like I'm not the first to be confused about this.

Cc: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: Quentin Schulz <quentin.schulz@theobroma-systems.com>
Cc: Ross Burton <ross.burton@arm.com>
Cc: Nicolas Dechesne <nicolas.dechesne@linaro.org>
Signed-off-by: Johan Korsnes <johan.korsnes@remarkable.no>
---
 .../doc/bitbake-user-manual/bitbake-user-manual-metadata.rst   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Quentin Schulz Sept. 29, 2022, 12:15 p.m. UTC | #1
Hi Johan,

On 9/29/22 2:11 PM, Johan Korsnes <johan.korsnes@remarkable.no> wrote:
> Inform the reader that there should be no need for spaces in the value
> when using removal override `:remove`.
> 
> Considering why spaces are used in the other override operators, it
> might seem obvious that they aren't needed for the removal operator.
> But, it seems like I'm not the first to be confused about this.
> 
> Cc: Richard Purdie <richard.purdie@linuxfoundation.org>
> Cc: Quentin Schulz <quentin.schulz@theobroma-systems.com>
> Cc: Ross Burton <ross.burton@arm.com>
> Cc: Nicolas Dechesne <nicolas.dechesne@linaro.org>
> Signed-off-by: Johan Korsnes <johan.korsnes@remarkable.no>
> ---
>   .../doc/bitbake-user-manual/bitbake-user-manual-metadata.rst   | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
> index 337821612c..0cc3ba18eb 100644
> --- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
> +++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
> @@ -330,7 +330,8 @@ Removal (Override Style Syntax)
>   
>   You can remove values from lists using the removal override style
>   syntax. Specifying a value for removal causes all occurrences of that
> -value to be removed from the variable.
> +value to be removed from the variable. Unlike ":append" and ":prepend",
> +it should never be necessary to pre/post-fix the value with a space.
>   

s/it should never be/it is not/

or "there is no need to add a leading or trailing space to the value"?

Cheers,
Quentin

>   When you use this syntax, BitBake expects one or more strings.
>   Surrounding spaces and spacing are preserved. Here is an example::
>
diff mbox series

Patch

diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
index 337821612c..0cc3ba18eb 100644
--- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
+++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-metadata.rst
@@ -330,7 +330,8 @@  Removal (Override Style Syntax)
 
 You can remove values from lists using the removal override style
 syntax. Specifying a value for removal causes all occurrences of that
-value to be removed from the variable.
+value to be removed from the variable. Unlike ":append" and ":prepend",
+it should never be necessary to pre/post-fix the value with a space.
 
 When you use this syntax, BitBake expects one or more strings.
 Surrounding spaces and spacing are preserved. Here is an example::