3 comments:
File Documentation/technotes/2020-03-unit-testing-coreboot.md:
toolchain […]
Done
Compiler for the host
_must_ support the same language standards as the target compiler.
This needs to be the first sentence in this section, as it is a requirement. […]
Yes, I should change the order of sentences in this paragraph. I kept the description of nice-to-have here for simplicity, no need for extra bullet under "Desirables".
* Same language for tests and code
Unit tests will be written in C, because coreboot code is also written in C
This might be a requirement, not a desirable. […]
Actually I was thinking exactly about this, when moving this bullet from Requirements here - don't want to have a must-have which is broken by GoogleTest immediately. But let's keep it simple and follow your suggestion. After all this _requirement_ is the main reason why we don't choose GoogleTest.
To view, visit change 39893. To unsubscribe, or for help writing mail filters, visit settings.