On Wed, Nov 29, 2023 at 1:13 PM Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx> wrote: > > On (23/11/28 23:19), Masahiro Yamada wrote: > > [..] > > > KCONFIG_WERROR is meant to turn all warnings > > to errors. > > I do not see getenv("KCONFIG_WERROR") > > sprinkled everywhere in Kconfig. > > One more thing, you cannot directly exit(1) > > from sym_calc_value(). > > We do exit(1) for KCONFIG_WARN_UNKNOWN_SYMBOLS in conf_read(). > > I can introduce two new helpers that will tell if confdata.c and symbol.c > triggered any warnings and if KCONFIG_WERROR is set. And then different > code paths can call them and handle exit gracefully, depending on the > context (ncurses, menu, etc.). > > Something like this I do not want to patch warnings one by one. I will take some time to think about it. -- Best Regards Masahiro Yamada