Re: [PATCH v3 1/8] build: split dependencies into mandatory and optional

Burakov, Anatoly
2024-09-09T08:41:44Z

On 9/6/2024 4:51 PM, Bruce Richardson wrote:
> On Wed, Sep 04, 2024 at 04:08:29PM +0100, Anatoly Burakov wrote:
>> Allow specifying dependencies as either mandatory or optional. This does
>> not change anything about the build, but it is useful for tooling to know
>> if a dependency is required or not.
>>
>
> Ideally we probably want to merge the support in this patch with the idea
> expressed in [1]
>
> /Bruce
>
> [1] https://patches.dpdk.org/project/dpdk/patch/20231220142152.492556-4-bruce.richardson@intel.com/

Hi Bruce,

Please correct me if I'm wrong, but doesn't it essentially do the same
thing? You're adding a new loop over optional deps, while I'm simply
extending existing loop to iterate over deps + optional deps, which
would achieve the same thing?

What does that patch do that this patch doesn't?

--
Thanks,
Anatoly