From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C5650C2D0DE for ; Wed, 1 Jan 2020 21:04:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 97E562064B for ; Wed, 1 Jan 2020 21:04:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727309AbgAAVEd (ORCPT ); Wed, 1 Jan 2020 16:04:33 -0500 Received: from zeniv.linux.org.uk ([195.92.253.2]:58440 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725871AbgAAVEd (ORCPT ); Wed, 1 Jan 2020 16:04:33 -0500 Received: from viro by ZenIV.linux.org.uk with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1imlAI-000GeY-2c; Wed, 01 Jan 2020 21:04:26 +0000 Date: Wed, 1 Jan 2020 21:04:26 +0000 From: Al Viro To: Arvind Sankar Cc: Masahiro Yamada , linux-kbuild@vger.kernel.org, Rob Landley , Randy Dunlap , linux-kernel@vger.kernel.org Subject: Re: [PATCH] menuconfig: restore prompt dependencies in help text Message-ID: <20200101210426.GA8904@ZenIV.linux.org.uk> References: <20191231055839.GG4203@ZenIV.linux.org.uk> <20200101204152.402906-1-nivedita@alum.mit.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200101204152.402906-1-nivedita@alum.mit.edu> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 01, 2020 at 03:41:52PM -0500, Arvind Sankar wrote: > Commit bcdedcc1afd6 ("menuconfig: print more info for symbol without > prompts") moved some code from get_prompt_str to get_symbol_str so that > dependency information for symbols without prompts could be shown. > > This code would be better copied rather than moved, as the change had > the side-effect of not showing any extra dependencies that the prompt > might have over the symbol. > > Put back a copy of the dependency printing code in get_prompt_str. Umm... Is "visible" really accurate in this case? AFAICS, the entry (and help for it) _is_ visible with EXPERT=n. OTOH, with EXPERT=y and MULTIUSER=n it disappears completely. I'm not familiar with kconfig guts (and not too concerned about that feature of help there, TBH), but it looks like what you are printing there is some mix of dependencies ("visible when") and selectability...