Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp16764351ybl; Wed, 1 Jan 2020 13:07:07 -0800 (PST) X-Google-Smtp-Source: APXvYqyuuGWQ9t5s65Gzb1LTX3/d5hPV6JfJZaDrz+jM+8bLpd1/SbX4WC0AH893jOFrsA3dK5gq X-Received: by 2002:a9d:22:: with SMTP id 31mr83171259ota.173.1577912827117; Wed, 01 Jan 2020 13:07:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577912827; cv=none; d=google.com; s=arc-20160816; b=OZMxRhV5gaBDD9/xIYzLPp9gsVZVyGxnIubsr8J81qceuUSYKATBhyy/Cwnx/vZMJQ OP/sKgI8gXz5/XYE6i13EOa/USmY6Xyp2hGdqeraRlWz+qX06xdVkQB2GEypyvUmLtuL HYDmeEwkiGaiMKPWi9/MkPiWCq++HVD7I8bef0vI5jsStLPsioCD/FiRTssh/a+G52R1 WUwx+GE7L2cCAEHEUZGx7nIlSrMHuTVna9GSiF4QuoiuPtX8KUGu8Wp83SynZN49OvU1 EllNzG0lClziIhvpmB+wNk8+QxQRozPz2AvD0R0EN+SiuPfaAerHQ2KjlrKD7I7T4hfu vNrg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date; bh=hEbBnQ5MuKcdfnCb+A1UohuCg9woJJMiBIt3P7rdqDg=; b=veivpg8ly5dGkLMqYRL2uAmWjfriiIjOhyzlbQAur7pPHFks+R1d0kVAoRqtmz5o5n +2KsiJIoXXJs7LVsP4qKe9Om+f+Lu/oIiRI10OJ4PoGSSvAYgFBOn5VEIpG5Pq8ZajCO oS3xUWv8Sroo5knep69hhcm9vEgAPGvH7KgElJ79LHpdCmOCPgZsGrDOGM8HNJBIhU87 L8oj7/qM+jyjlX1Y5ZiLfx8YptWDjvpp6uCizNJuVH8QDellmt1Rl4ceiBk59JNyUZBV O6kNeLHIUBaph2+/IA83SyleLjHsKuGAB+Eho+RZWiKbS5sWOrGhNCX1XC0gcbb+1mw0 pL8A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id i19si16138880oik.272.2020.01.01.13.06.40; Wed, 01 Jan 2020 13:07:07 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727289AbgAAVEd (ORCPT + 99 others); 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...