[PATCH 2/2] qconfig: Ignore defconfigs containing #include

[Simon Glass]

Hi Tom,

On Sun, 13 Oct 2024 at 10:15, Tom Rini <trini at konsulko.com> wrote:
>
> On Sat, Oct 12, 2024 at 05:22:30PM -0600, Simon Glass wrote:
>
> > Running "tools/qconfig.py -s" will re-sync files with #include in them
> > and so un-#include them.
> >
> > Ignore these and mark them as failures.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > Fixes: https://source.denx.de/u-boot/custodians/u-boot-dm/-/issues/28
>
> How about marking them as warnings? It may be good to re-sync
> configs/am62px_evm_a53_defconfig for example and then re-add the
> #include line, but configs/am62x_evm_a53_ethboot_defconfig on the other
> hand very much should not be resynced. Thanks for getting on this issue
> so quickly.

Do you mean still do the resync, but show a warning? It does
obliterate the old defconfig, but I suppose people can just not commit
the changes. Is that what you are thinking?

Regards,
SImon