[U-Boot] [PATCH 5/5] check-config: allow to complete build even with ad-hoc CONFIG options
Simon Glass
sjg at chromium.org
Tue Sep 27 02:34:23 CEST 2016
Hi Masahiro,
On 25 September 2016 at 22:05, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> Currently, the check-config.sh terminates the build when unknown
> ad-hoc options are detected. I think it is too much because we may
> want to patch config headers locally in a build/deployment project.
>
> So, let's relax check-config.sh to just warn even if it detects
> options that are not in the whitelist. Instead, this check can be
> done at the end of build, along with other checks. It will catch
> more attention.
>
> Even with this change, the Buildman tool catches new warnings,
> so Tom can give NACK to new ad-hoc options.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
> scripts/check-config.sh | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
I am not sure this is a good idea. We need buildman to fail the build;
otherwise patches will make it to mainline with new CONFIGs. Perhaps
we should have an ENV variable to skip the check?
Regards,
Simon
More information about the U-Boot
mailing list