Name of user not set #1002873 has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39894 )
Change subject: tests: Add build subsystem for unit testing coreboot ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39894/1/tests/include/config.h File tests/include/config.h:
PS1:
I'm not really sure I understand the dependency problem you're describing? You just need to make the […]
Actually at the time of writing above, I was unaware of the easy method of defining variable per-target, which I'm now using even in my Makefile.inc. So if we can define per-target variables for Kconfig to consume, then a little Makefile magic should allow me to create what I need.
However, we will need to create config.h for every test right? I'm afraid this may take a significant time to run (for the first, clean build).
That being said, I would like to propose to postpone this work for future, if this is acceptable. This shouldn't be a killer at least at the beginning of writing tests. Let's stabilize what we have now and then introduce changes on top of it.