Re: RFC: Behavior of make *config

Patrick Georgi via coreboot <[email protected]>
Newsgroups gmane.linux.bios
Message-ID <[email protected]>
29. November 2023 12:14, "Poeche, Uwe via coreboot" <[email protected]> schrieb:
> Your above mentioned patch helps (I'm a reviewer of this). When I tried I also noticed that this
> does not point out loudly if you have problems in your config. So the approach without the patch
> (in result more strict) is from my opinion better if we have a workaround beside edit config
> manually. In the parallel thread here Martin mentioned the same workaround as you mentioned on IRC
> on Monday. But I get not work.

Right, because the code checks for the variables' presence, not for their value. I was mistaken on that, as was Martin apparently.

I modified the changeset so that it still doesn't fail on warnings during *config, but still prints them. That doesn't matter a whole lot with menuconfig and nconfig because they proceed to clear the screen immediately after, but oldconfig and olddefconfig are somewhat more helpful now:

$ make oldconfig
/home/pgeorgi/coreboot/.config:152:warning: unknown symbol: VBT_DATA_SIZE_KB
*
* Restart config...
*
...

$ make olddefconfig
/home/pgeorgi/coreboot/.config:152:warning: unknown symbol: VBT_DATA_SIZE_KB
#
# configuration written to /home/pgeorgi/coreboot/.config
#

Does that help?


Patrick
_______________________________________________
coreboot mailing list -- [email protected]
To unsubscribe send an email to [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.