Re: [PATCH] scripts: merge_config: Add flag to prevent unsetting config option

From: Rahul Rameshbabu
Date: Mon Aug 07 2023 - 00:13:23 EST



On Sun, 06 Aug, 2023 23:19:55 +0900 "Masahiro Yamada" <masahiroy@xxxxxxxxxx> wrote:
> On Sun, Jul 30, 2023 at 6:42 AM Rahul Rameshbabu
> <sergeantsagara@xxxxxxxxxxxxxx> wrote:
>>
>> Overriding a previously defined entry for a config option with 'is not set'
>> may be undesirable in some fragment configuration setups.
>
> Then, you should remove the 'is not set' entry from the fragment.

I had a feeling that was the expectation. Just for reference, my flow
for generating fragments looks like the following.

1. make allnoconfig
2. make menuconfig # select the options that I desire for the fragment

We can drop this patch if this is the expected developer flow. I
realized that overriding with 'is not set' entries in a fragment is
likely desirable, so I made this behavior change as part of a flag that
can be set by the user.

--
Thanks,

Rahul Rameshbabu