Re: [docs] [PATCH] ref-manual: faq: add q&a on class appends

["Antonin Godard" <antonin.godard@bootlin.com>]

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