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

From: Masahiro Yamada
Date: Mon Aug 07 2023 - 15:05:18 EST


On Mon, Aug 7, 2023 at 1:13 PM Rahul Rameshbabu
<sergeantsagara@xxxxxxxxxxxxxx> wrote:
>
>
> 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


Sorry, I could not understand
how these steps generate a fragment file.

You will get a full .config file
after 'make menuconfig'.





> 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
>


--
Best Regards
Masahiro Yamada