[U-Boot] [U-Boot, 3/4] tools: moveconfig: warn loudly if moved option has no entry in Kconfig

Tom Rini
2016-09-07T17:58:05Z

On Mon, Aug 22, 2016 at 10:18:21PM +0900, Masahiro Yamada wrote:

> Currently, the tool gives up moving an option quietly if its entry
> was not found in Kconfig.
>
> If the option is not defined in the config header in the first
> place, it is no problem (as the Kconfig entry may have been hidden
> by reasonable "depends on").
>
> However, if the option is defined in the config header, the missing
> Kconfig entry is a sign of possible behavior change. It is highly
> recommended to manually check if the option has been moved as
> expected. In this case, let's add "suspicious" in the log and
> change the log color (if --color option is given) to make it stand
> out.
>
> This was suggested by Tom in [1].
>
> [1] http://lists.denx.de/pipermail/u-boot/2016-July/261988.html
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Suggested-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160907/b1200012/attachment.sig>