* [PATCH] ref-manual: faq: add q&a on class appends
@ 2024-10-24 9:14 Antonin Godard
2024-10-24 9:27 ` [docs] " Quentin Schulz
2024-10-24 10:31 ` Richard Purdie
0 siblings, 2 replies; 4+ messages in thread
From: Antonin Godard @ 2024-10-24 9:14 UTC (permalink / raw)
To: docs; +Cc: Richard Purdie, Thomas Petazzoni, Antonin Godard
Add the suggestion by Richard to the FAQ, with some minor rewordings and
additional punctuations.
Suggested-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
---
documentation/ref-manual/faq.rst | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
index bab284bbfdefae02c284204d3c94891a8238239a..3dcb1f862e24716be703f9fd3e811d2fb9ed34a0 100644
--- a/documentation/ref-manual/faq.rst
+++ b/documentation/ref-manual/faq.rst
@@ -259,6 +259,25 @@ Within the :term:`Build Directory`, is the ``tmp`` directory. To remove all the
build output yet preserve any source code or downloaded files from
previous builds, simply remove the ``tmp`` directory.
+Why isn't there a way to append bbclass files like bbappend for recipes?
+------------------------------------------------------------------------
+
+The Yocto Project has consciously chosen not to implement such functionality.
+Class code is designed to be shared and reused, and expose some level of
+configuration to its users. We want to encourage people to share these changes
+so we can build the best classes.
+
+If the ``append`` functionality was available for classes, our evidence and
+experience suggests that people would create their custom changes in their
+layer instead of sharing and discussing the issues and/or limitations they
+encountered. This would lead to bizarre class interactions when new layers are
+included. We therefore consciously choose to have a natural pressure to share
+class code improvements or fixes.
+
+There are also technical considerations like which recipes a class append would
+apply to and how that would fit within the layer model. These are complications
+we think we can live without!
+
Customizing generated images
============================
---
base-commit: 72bbff33fdce60c133074b4ac09721c4ddac5372
change-id: 20241024-faq-class-append-88bcb9536b39
Best regards,
--
Antonin Godard <antonin.godard@bootlin.com>
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [docs] [PATCH] ref-manual: faq: add q&a on class appends
2024-10-24 9:14 [PATCH] ref-manual: faq: add q&a on class appends Antonin Godard
@ 2024-10-24 9:27 ` Quentin Schulz
2024-10-24 9:29 ` Antonin Godard
2024-10-24 10:31 ` Richard Purdie
1 sibling, 1 reply; 4+ messages in thread
From: Quentin Schulz @ 2024-10-24 9:27 UTC (permalink / raw)
To: antonin.godard, docs; +Cc: Richard Purdie, Thomas Petazzoni
Hi Antonin,
On 10/24/24 11:14 AM, Antonin Godard via lists.yoctoproject.org wrote:
> Add the suggestion by Richard to the FAQ, with some minor rewordings and
> additional punctuations.
>
> Suggested-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> ---
> documentation/ref-manual/faq.rst | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> index bab284bbfdefae02c284204d3c94891a8238239a..3dcb1f862e24716be703f9fd3e811d2fb9ed34a0 100644
> --- a/documentation/ref-manual/faq.rst
> +++ b/documentation/ref-manual/faq.rst
> @@ -259,6 +259,25 @@ Within the :term:`Build Directory`, is the ``tmp`` directory. To remove all the
> build output yet preserve any source code or downloaded files from
> previous builds, simply remove the ``tmp`` directory.
>
> +Why isn't there a way to append bbclass files like bbappend for recipes?
> +------------------------------------------------------------------------
> +
> +The Yocto Project has consciously chosen not to implement such functionality.
> +Class code is designed to be shared and reused, and expose some level of
s/expose/exposes/
except if we meant "class code is designed to expose" rather than "class
code exposes"
> +configuration to its users. We want to encourage people to share these changes
> +so we can build the best classes.
> +
> +If the ``append`` functionality was available for classes, our evidence and
> +experience suggests that people would create their custom changes in their
s/suggests/suggest/ (evidence+experience)
Otherwise looks good to me :)
Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
Cheers,
Quentin
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [docs] [PATCH] ref-manual: faq: add q&a on class appends
2024-10-24 9:27 ` [docs] " Quentin Schulz
@ 2024-10-24 9:29 ` Antonin Godard
0 siblings, 0 replies; 4+ messages in thread
From: Antonin Godard @ 2024-10-24 9:29 UTC (permalink / raw)
To: Quentin Schulz, docs; +Cc: Richard Purdie, Thomas Petazzoni
Hi Quentin,
On Thu Oct 24, 2024 at 11:27 AM CEST, Quentin Schulz wrote:
> Hi Antonin,
>
> On 10/24/24 11:14 AM, Antonin Godard via lists.yoctoproject.org wrote:
>> Add the suggestion by Richard to the FAQ, with some minor rewordings and
>> additional punctuations.
>>
>> Suggested-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
>> ---
>> documentation/ref-manual/faq.rst | 19 +++++++++++++++++++
>> 1 file changed, 19 insertions(+)
>>
>> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
>> index bab284bbfdefae02c284204d3c94891a8238239a..3dcb1f862e24716be703f9fd3e811d2fb9ed34a0 100644
>> --- a/documentation/ref-manual/faq.rst
>> +++ b/documentation/ref-manual/faq.rst
>> @@ -259,6 +259,25 @@ Within the :term:`Build Directory`, is the ``tmp`` directory. To remove all the
>> build output yet preserve any source code or downloaded files from
>> previous builds, simply remove the ``tmp`` directory.
>>
>> +Why isn't there a way to append bbclass files like bbappend for recipes?
>> +------------------------------------------------------------------------
>> +
>> +The Yocto Project has consciously chosen not to implement such functionality.
>> +Class code is designed to be shared and reused, and expose some level of
>
> s/expose/exposes/
>
> except if we meant "class code is designed to expose" rather than "class
> code exposes"
Agreed it should probably by exposes here.
>> +configuration to its users. We want to encourage people to share these changes
>> +so we can build the best classes.
>> +
>> +If the ``append`` functionality was available for classes, our evidence and
>> +experience suggests that people would create their custom changes in their
>
> s/suggests/suggest/ (evidence+experience)
Good catch. :)
> Otherwise looks good to me :)
>
> Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
>
> Cheers,
> Quentin
Thank you!
Antonin
--
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ref-manual: faq: add q&a on class appends
2024-10-24 9:14 [PATCH] ref-manual: faq: add q&a on class appends Antonin Godard
2024-10-24 9:27 ` [docs] " Quentin Schulz
@ 2024-10-24 10:31 ` Richard Purdie
1 sibling, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2024-10-24 10:31 UTC (permalink / raw)
To: Antonin Godard, docs; +Cc: Thomas Petazzoni
On Thu, 2024-10-24 at 11:14 +0200, Antonin Godard wrote:
> Add the suggestion by Richard to the FAQ, with some minor rewordings and
> additional punctuations.
>
> Suggested-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> ---
> documentation/ref-manual/faq.rst | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> index bab284bbfdefae02c284204d3c94891a8238239a..3dcb1f862e24716be703f9fd3e811d2fb9ed34a0 100644
> --- a/documentation/ref-manual/faq.rst
> +++ b/documentation/ref-manual/faq.rst
> @@ -259,6 +259,25 @@ Within the :term:`Build Directory`, is the ``tmp`` directory. To remove all the
> build output yet preserve any source code or downloaded files from
> previous builds, simply remove the ``tmp`` directory.
>
> +Why isn't there a way to append bbclass files like bbappend for recipes?
> +------------------------------------------------------------------------
> +
> +The Yocto Project has consciously chosen not to implement such functionality.
> +Class code is designed to be shared and reused, and expose some level of
> +configuration to its users. We want to encourage people to share these changes
> +so we can build the best classes.
> +
> +If the ``append`` functionality was available for classes, our evidence and
> +experience suggests that people would create their custom changes in their
> +layer instead of sharing and discussing the issues and/or limitations they
> +encountered. This would lead to bizarre class interactions when new layers are
> +included. We therefore consciously choose to have a natural pressure to share
> +class code improvements or fixes.
> +
> +There are also technical considerations like which recipes a class append would
> +apply to and how that would fit within the layer model. These are complications
> +we think we can live without!
> +
> Customizing generated images
> ============================
>
Looks good, thanks!
If/as/when I see anything else which should be in there I'll try and
note something down. We should really try and collect more of these.
Cheers,
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-10-24 10:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-24 9:14 [PATCH] ref-manual: faq: add q&a on class appends Antonin Godard
2024-10-24 9:27 ` [docs] " Quentin Schulz
2024-10-24 9:29 ` Antonin Godard
2024-10-24 10:31 ` Richard Purdie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox