Re: [PATCH] kconfig: Warn if there is more than one help text

From: Masahiro Yamada
Date: Sat Jan 13 2018 - 08:19:49 EST


2018-01-12 15:47 GMT+09:00 Ulf Magnusson <ulfalizer@xxxxxxxxx>:
> 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 <ulfalizer@xxxxxxxxx>
> ---

Applied to linux-kbuild/kconfig
replacing "kconfig: Don't leak help strings during parsing"

Thanks!

--
Best Regards
Masahiro Yamada