[PATCH v2 1/3] buildman: allow specifying configuration fragments

[Quentin Schulz]

Hi Simon,

On 4/18/25 1:39 PM, Simon Glass wrote:
> Hi Quentin,
> 
> On Fri, 18 Apr 2025 at 05:27, Quentin Schulz <quentin.schulz at cherry.de> wrote:
>>
>> Hi Heinrich,
>>
>> On 4/17/25 12:40 AM, Heinrich Schuchardt wrote:
>>> Currently we are no able to build with configuration fragments in our CI.
>>> With this patch buildman gets a new argument --fragments for passing a
>>> comma separated list of configuration fragments to add to the board
>>> defconfigs, e.g.
>>>
>>>       tools/buildman/buildman \
>>>       -o build \
>>>       -k qemu-riscv64_smode \
>>>       --fragments acpi.config
>>>
>>
>> What about using:
>>
>> --fragment acpi.config --fragment fragment2.config
>>
>> ?
> 
> Yes that could be useful. It would also allow wildcards, although the
> code would need to drop the config/ directory prefix. Also, I see that
> -z is available so perhaps that could be a (bad) short option?
> 

I really dislike short options, but whatever floats your boat :)

> Another idea would be to also allow specifying the fragment name
> without the '.config' suffix. Less to type.
> 

It's actually a fair point, we do already strip the _defconfig suffix, 
and it seems config fragments are necessarily suffixed by .config, so I 
believe this is fair.

Cheers,
Quentin