From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755150AbeAMNTo (ORCPT + 1 other); Sat, 13 Jan 2018 08:19:44 -0500 Received: from conssluserg-03.nifty.com ([210.131.2.82]:26571 "EHLO conssluserg-03.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754934AbeAMNTm (ORCPT ); Sat, 13 Jan 2018 08:19:42 -0500 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-03.nifty.com w0DDJc7B030654 X-Nifty-SrcIP: [209.85.217.170] X-Google-Smtp-Source: ACJfBos4g68xumYMdL/1Dp3NOCkd8Rb1WaQ8rL1oikPT8yvZqm6JpLSmLW8vPjwWIFEslvUDk1kY5JKt7VTah53xTcg= MIME-Version: 1.0 In-Reply-To: <20180112064747.22854-1-ulfalizer@gmail.com> References: <20180112064747.22854-1-ulfalizer@gmail.com> From: Masahiro Yamada Date: Sat, 13 Jan 2018 22:18:57 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] kconfig: Warn if there is more than one help text To: Ulf Magnusson Cc: Linux Kbuild mailing list , Sam Ravnborg , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: 2018-01-12 15:47 GMT+09:00 Ulf Magnusson : > Avoids mistakes like in the following real-world example, where only the > final help string ("Say Y...") was used. This particular example was > fixed in commit 561b29e4ec8d ("media: fix media Kconfig help syntax > issues"). > > config DVB_NETUP_UNIDVB > ... > select DVB_CXD2841ER if MEDIA_SUBDRV_AUTOSELECT > ---help--- > Support for NetUP PCI express Universal DVB card. > help > Say Y when you want to support NetUP Dual Universal DVB card > ... > > This now prints the following warning: > > drivers/media/pci/netup_unidvb:13: warning: 'DVB_NETUP_UNIDVB' defined with more than one help text -- only the last one will be used > > Also free() any extra help strings. > > Signed-off-by: Ulf Magnusson > --- Applied to linux-kbuild/kconfig replacing "kconfig: Don't leak help strings during parsing" Thanks! -- Best Regards Masahiro Yamada