Re: [OE-core] [PATCH v2 0/2] add basic b4 config file

[Quentin Schulz <quentin.schulz@cherry.de>]

Hi Khem,

On 2/3/25 8:42 PM, Khem Raj wrote:
> On Mon, Feb 3, 2025 at 7:18 AM Quentin Schulz via
> lists.openembedded.org <foss=0leil.net@lists.openembedded.org> wrote:
>>
>> This adds very basic support for using b4 in OE-Core. This essentially
>> just automatically adds the proper mailing list when running b4 prep
>> --auto-to-cc.
>>
>> This also adds the b4 wrapper script that poky will need to use in order
>> to identify which mailing list a patch needs to be sent to and do some
>> additional per-patch checks that aren't necessary when running from
>> vanilla OE-Core.
>>
>> Once this is merged, poky will need to merge OE-Core into itself and
>> override with its own .b4-config that makes use of that script.
>>
>> Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de>
>> ---
>> Changes in v2:
>> - add mention to first patch that running patchtest for b4 prep --check
>>    is what we're eventually aiming for,
>> - add poky b4 wrapper script (it used to be in the patch series for
>>    poky, c.f. https://lore.kernel.org/poky/20250123-b4-support-v1-1-4841f52eccbb@cherry.de/T/#m95d6aa25cc42352416da477cf07d7b3cbd671231
>> - poky b4 wrapper script properly identifies patches with files modified
>>    in both OE-Core and other repos part of poky,
>> - poky b4 wrapper script pretty prints when there's more than one
>>    project modified by a script, instead of lazily printing a Python
>>    list,
>> - Link to v1: https://lore.kernel.org/r/20250117-b4-support-v1-1-fe74a5f50eb6@cherry.de
>>
>> ---
>> Quentin Schulz (2):
>>        add basic b4 config file
>>        scripts: add b4-wrapper for poky
>>
>>   .b4-config                 |   3 +
>>   scripts/b4-wrapper-poky.py | 170 +++++++++++++++++++++++++++++++++++++++++++++
> 
> is there anything specific to poky in this script ? if not then it
> should not use that in name, it will

Yes it is poky specific...

> make it confusing. In oe-core we do not refer to particular distros if
> not needed.
> 

...git-repo poky specific, not poky distro specific. Do you have 
something better to suggest than this?

Cheers,
Quentin