Hi,
On Thu, Dec 16, 2010 at 4:26 PM, Michal Marek <mmarek@xxxxxxx> wrote:
> On Thu, Dec 09, 2010 at 08:11:38AM +0000, Jan Beulich wrote:
>> This lead to non-selected, non-user-selectable options to be written
>> out to .config. This is not only pointless, but also preventing the
>> user to be prompted should any of those options eventually become
>> visible (e.g. by de-selecting the *_AUTO options the "visible"
>> attribute was added for.
>>
>> Furthermore it is quite logical for the "visible" attribute of a menu
>> to control the visibility of all contained prompts, which is what the
>> patch does.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
>
> It also fixes the issue that all{mod,yes}config set some options from
> the invisible menus that cannot be selected interactively (e.g.
> I2C_ALGOPCF, that is only selected by I2C_ELEKTOR on !SMP), so this
> patch is
>
> Acked-by: Michal Marek <mmarek@xxxxxxx>
>
> unless Arnaud has a good reason not to take it.
>
Actually, I wanted to keep the parsing and tree construction as simple
as possible. Fix-up like that should really happen in menu_finalize()
as we will anyway traverse all the menus once again there and it is
already doing all kind of dependency simplification, so that would
keep everything at the same place.
Say that if I do not show up with a version which would move this in
menu_finalize() by the end of the week (sooner if you want that for
.37), feel free to forget about this mail. Hopefully, I'll not move
back to Linux this week-end, so I should have the time to do that.
- Arnaud
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
